Skip to content

Commit

Permalink
Fix flaky integration test failures (#7262)
Browse files Browse the repository at this point in the history
This partially reverts commit 20b1211,
which was landed in #7254.
Specifically, it reverts the addition of "noWaitForReady" to the
health-checker's gRPC config. This appears to stop the flaky `last
resolver error: produced zero addresses` failures we've been seeing in
the CI integration tests.
  • Loading branch information
aarongable authored Jan 16, 2024
1 parent a9a87cd commit d38b7b6
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
3 changes: 1 addition & 2 deletions test/config-next/health-checker.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
{
"grpc": {
"timeout": "1s",
"noWaitForReady": true
"timeout": "1s"
},
"tls": {
"caCertFile": "test/grpc-creds/minica.pem",
Expand Down
3 changes: 1 addition & 2 deletions test/config/health-checker.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
{
"grpc": {
"timeout": "1s",
"noWaitForReady": true
"timeout": "1s"
},
"tls": {
"caCertFile": "test/grpc-creds/minica.pem",
Expand Down

0 comments on commit d38b7b6

Please sign in to comment.