From d38b7b685ba62d494d72822fbd9e7ed6e9e31cd5 Mon Sep 17 00:00:00 2001 From: Aaron Gable Date: Tue, 16 Jan 2024 09:50:13 -0800 Subject: [PATCH] Fix flaky integration test failures (#7262) This partially reverts commit 20b121138cd30a86a32cce86d7266bd71d1da5b5, which was landed in https://github.com/letsencrypt/boulder/pull/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. --- test/config-next/health-checker.json | 3 +-- test/config/health-checker.json | 3 +-- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/test/config-next/health-checker.json b/test/config-next/health-checker.json index 00183a888..599916264 100644 --- a/test/config-next/health-checker.json +++ b/test/config-next/health-checker.json @@ -1,7 +1,6 @@ { "grpc": { - "timeout": "1s", - "noWaitForReady": true + "timeout": "1s" }, "tls": { "caCertFile": "test/grpc-creds/minica.pem", diff --git a/test/config/health-checker.json b/test/config/health-checker.json index 00183a888..599916264 100644 --- a/test/config/health-checker.json +++ b/test/config/health-checker.json @@ -1,7 +1,6 @@ { "grpc": { - "timeout": "1s", - "noWaitForReady": true + "timeout": "1s" }, "tls": { "caCertFile": "test/grpc-creds/minica.pem",