Skip to content

Conversation

giorio94
Copy link
Member

The blamed commit replaced the default wildcard toleration associated with the Cilium Operator with a precise set of keys. However, the GKE workflows configure a different toleration, hence we need to explicitly include it as part of the tolerations list.

Fixes: 589164b ("Adding taint to block deployment to drained nodes")

The blamed commit replaced the default wildcard toleration associated
with the Cilium Operator with a precise set of keys. However, the GKE
workflows configure a different toleration, hence we need to explicitly
include it as part of the tolerations list.

Fixes: 589164b ("Adding taint to block deployment to drained nodes")
Signed-off-by: Marco Iorio <marco.iorio@isovalent.com>
@giorio94 giorio94 added area/CI Continuous Integration testing issue or flake release-note/ci This PR makes changes to the CI. labels Jul 14, 2025
@giorio94 giorio94 added the needs-backport/1.18 This PR / issue needs backporting to the v1.18 branch label Jul 14, 2025
@giorio94
Copy link
Member Author

/ci-gke

@giorio94
Copy link
Member Author

/net-perf-gke

@giorio94
Copy link
Member Author

/test

@giorio94 giorio94 marked this pull request as ready for review July 14, 2025 08:44
@giorio94 giorio94 requested review from a team as code owners July 14, 2025 08:44
@aanm aanm added dont-merge/bad-bot To prevent MLH from marking ready-to-merge. and removed dont-merge/bad-bot To prevent MLH from marking ready-to-merge. labels Jul 14, 2025
@aanm aanm enabled auto-merge July 14, 2025 09:04
@aanm aanm added this pull request to the merge queue Jul 14, 2025
Merged via the queue into main with commit b02b392 Jul 14, 2025
105 checks passed
@aanm aanm deleted the pr/giorio94/main/unbreak-gke-tests branch July 14, 2025 09:13
@giorio94
Copy link
Member Author

Dropped the needs-backport label, as superseded by #40531

@giorio94 giorio94 removed the needs-backport/1.18 This PR / issue needs backporting to the v1.18 branch label Jul 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/CI Continuous Integration testing issue or flake release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants