Skip to content

Comments

Revert "increase ready for server channel timeout for outlier_detection , custom_lb, round_robin test"#225

Open
sergiitk wants to merge 1 commit intomainfrom
revert-222-lds_increase_timeout
Open

Revert "increase ready for server channel timeout for outlier_detection , custom_lb, round_robin test"#225
sergiitk wants to merge 1 commit intomainfrom
revert-222-lds_increase_timeout

Conversation

@sergiitk
Copy link
Member

@sergiitk sergiitk commented Feb 6, 2026

Reverts #222

We didn't notice improvements the flakiness rate of these tests.

Context: b/459985396#comment48

@sergiitk sergiitk requested a review from a team as a code owner February 6, 2026 23:11
@sergiitk
Copy link
Member Author

sergiitk commented Feb 6, 2026

For my week 2026-01-23 – 2026-01-30:

In fact, custom_lb_test and round_robin_test tests are in the the top 3.

$ # excluding grpc_xds_k8s_xbranch / grpc_xds_k8s_xlang
$ pbpaste| sort | uniq -c | sort -nr
   8 authz_test
   6 custom_lb_test
   5 round_robin_test
   4 remove_neg_test
   4 outlier_detection_test
   4 dualstack_test
   4 api_listener_test
   4 affinity_test
   3 security_test
   3 failover_test
   3 cloud_run_csm_inbound_test
   3 circuit_breaking_test
   3 app_net_test
   1 change_backend_service_test

@sergiitk
Copy link
Member Author

sergiitk commented Feb 6, 2026

@eshitachandwani - since you're investigating the issue, feel free to merge this when you're ready. Just in case you still need these tests running with extended timeouts for a bit longer.

@eshitachandwani
Copy link
Member

Will let the timeout be 10 mins for some more time till the investigation is going on.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants