Skip to content

Conversation

brlbil
Copy link
Contributor

@brlbil brlbil commented Aug 8, 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 the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Aug 8, 2025
@brlbil
Copy link
Contributor Author

brlbil commented Aug 8, 2025

/ci-gke

2 similar comments
@brlbil
Copy link
Contributor Author

brlbil commented Aug 8, 2025

/ci-gke

@brlbil
Copy link
Contributor Author

brlbil commented Aug 8, 2025

/ci-gke

@brlbil brlbil force-pushed the pr/brlbil/test-gke-failures branch 3 times, most recently from 7b70cf1 to 76aa443 Compare August 11, 2025 11:05
@brlbil
Copy link
Contributor Author

brlbil commented Aug 11, 2025

/ci-gke

2 similar comments
@brlbil
Copy link
Contributor Author

brlbil commented Aug 11, 2025

/ci-gke

@brlbil
Copy link
Contributor Author

brlbil commented Aug 11, 2025

/ci-gke

@brlbil brlbil force-pushed the pr/brlbil/test-gke-failures branch from 76aa443 to c035817 Compare August 11, 2025 13:44
@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-failures branch from c035817 to 04fabf3 Compare August 11, 2025 14:41
@brlbil
Copy link
Contributor Author

brlbil commented Aug 11, 2025

/ci-gke versions=all

@brlbil brlbil force-pushed the pr/brlbil/test-gke-failures branch from 04fabf3 to ef97b85 Compare August 12, 2025 07:59
@brlbil brlbil changed the title DO NOT MERGE: testing ci: reduce gke failures Aug 12, 2025
@brlbil brlbil added the release-note/ci This PR makes changes to the CI. label Aug 12, 2025
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Aug 12, 2025
@brlbil
Copy link
Contributor Author

brlbil commented Aug 12, 2025

/test

1 similar comment
@brlbil
Copy link
Contributor Author

brlbil commented Aug 12, 2025

/test

@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
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-failures branch from ef97b85 to 465d306 Compare August 12, 2025 13:54
@brlbil
Copy link
Contributor Author

brlbil commented Aug 12, 2025

/test

@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
@aanm aanm added this pull request to the merge queue Aug 13, 2025
Merged via the queue into main with commit 72ecadb Aug 13, 2025
306 of 307 checks passed
@aanm aanm deleted the pr/brlbil/test-gke-failures branch August 13, 2025 07:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge. 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