Skip to content

Conversation

brlbil
Copy link
Contributor

@brlbil brlbil commented Aug 11, 2025

The conformance-gke tests have been failing with the error message: error dialing backend: No agent available, as noted in the opened issue #41037. This PR changes the region, which does not completely solve the problem since the root cause is still unknown. However, after the change, the frequency of the same failure has significantly reduced.

Failures have been observed on the main and v1.18 branches, so I have tested the changes on both.

Test runs on the main branch:

Run 16879517490
Run 16882292979
Run 16883759631 (contains an unrelated failure)

Test runs on the v1.18 branch:

Run 16883023090
Run 16884801853 (contains an unrelated failure)

Additionally, there was a scheduled run using the old region that occurred around the same time as the tests above, and the failures were again observed:

Run 16883921129

@maintainer-s-little-helper maintainer-s-little-helper bot added backport/1.18 This PR represents a backport for Cilium 1.18.x of a PR that was merged to main. kind/backports This PR provides functionality previously merged into master. labels Aug 11, 2025
@brlbil
Copy link
Contributor Author

brlbil commented Aug 11, 2025

ci-gke

@brlbil
Copy link
Contributor Author

brlbil commented Aug 11, 2025

/ci-gke

1 similar comment
@brlbil
Copy link
Contributor Author

brlbil commented Aug 11, 2025

/ci-gke

@brlbil brlbil force-pushed the pr/brlbil/test-gke-ver branch from bffb271 to 3f96cd4 Compare August 11, 2025 15:17
@brlbil
Copy link
Contributor Author

brlbil commented Aug 11, 2025

/ci-gke versions=all

This commit changes default GKE test region from us-east1-c to us-west1-c,
due to test failure rate being hight.
This change reduces the number failures.

Signed-off-by: brlbil <birol@cilium.io>
@brlbil brlbil force-pushed the pr/brlbil/test-gke-ver branch from 3f96cd4 to a8c983a Compare August 12, 2025 07:56
@brlbil brlbil changed the title DO NOT MERGE: testing ci: reduce gke failures Aug 12, 2025
@brlbil
Copy link
Contributor Author

brlbil commented Aug 12, 2025

/test

@brlbil
Copy link
Contributor Author

brlbil commented Aug 12, 2025

/ci-eks

@brlbil brlbil marked this pull request as ready for review August 12, 2025 11:53
@brlbil brlbil requested review from a team as code owners August 12, 2025 11:53
Copy link
Member

@joestringer joestringer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for digging into this.

@joestringer joestringer added this pull request to the merge queue Aug 12, 2025
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Aug 12, 2025
Merged via the queue into v1.18 with commit 6b9ffb1 Aug 12, 2025
419 of 427 checks passed
@joestringer joestringer deleted the pr/brlbil/test-gke-ver branch August 12, 2025 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/1.18 This PR represents a backport for Cilium 1.18.x of a PR that was merged to main. kind/backports This PR provides functionality previously merged into master. ready-to-merge This PR has passed all tests and received consensus from code owners to merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants