From ac428c14fd8808e0066dc72f55e0ea5211a7763b Mon Sep 17 00:00:00 2001 From: Larry Safran Date: Mon, 29 Jan 2024 17:43:21 -0800 Subject: [PATCH] Relax some timeouts that have been flaky for the automated CI testing. (#10858) * Relax some timeouts that have been flaky for the automated CI testing. --- .../java/io/grpc/testing/integration/AbstractInteropTest.java | 4 ++-- .../src/test/java/io/grpc/testing/integration/RetryTest.java | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/interop-testing/src/main/java/io/grpc/testing/integration/AbstractInteropTest.java b/interop-testing/src/main/java/io/grpc/testing/integration/AbstractInteropTest.java index 75d9f03690..d450ece7bc 100644 --- a/interop-testing/src/main/java/io/grpc/testing/integration/AbstractInteropTest.java +++ b/interop-testing/src/main/java/io/grpc/testing/integration/AbstractInteropTest.java @@ -220,7 +220,7 @@ public abstract class AbstractInteropTest { * Constructor for tests. */ protected AbstractInteropTest() { - TestRule timeout = Timeout.seconds(60); + TestRule timeout = Timeout.seconds(90); try { timeout = new DisableOnDebug(timeout); } catch (Throwable t) { @@ -2212,7 +2212,7 @@ public abstract class AbstractInteropTest { } protected int operationTimeoutMillis() { - return 5000; + return 7000; } /** diff --git a/interop-testing/src/test/java/io/grpc/testing/integration/RetryTest.java b/interop-testing/src/test/java/io/grpc/testing/integration/RetryTest.java index 45ea303e5c..880320c6ca 100644 --- a/interop-testing/src/test/java/io/grpc/testing/integration/RetryTest.java +++ b/interop-testing/src/test/java/io/grpc/testing/integration/RetryTest.java @@ -244,7 +244,7 @@ public class RetryTest { private void assertRpcStatusRecorded( Status.Code code, long roundtripLatencyMs, long outboundMessages) throws Exception { - MetricsRecord record = clientStatsRecorder.pollRecord(5, SECONDS); + MetricsRecord record = clientStatsRecorder.pollRecord(7, SECONDS); TagValue statusTag = record.tags.get(RpcMeasureConstants.GRPC_CLIENT_STATUS); assertThat(statusTag.asString()).isEqualTo(code.toString()); assertThat(record.getMetricAsLongOrFail(DeprecatedCensusConstants.RPC_CLIENT_FINISHED_COUNT))