Skip to content

Commit 847896e

Browse files
bm1549devflow.devflow-routing-intake
andauthored
Fix log injection smoke test flakiness under CI load (#10999)
Fix log injection smoke test flakiness under CI load Increase BaseApplication.TIMEOUT_IN_NANOS from 10s to 30s to align with the test's PollingConditions timeout (also 30s). This is the only smoke test with a bidirectional handshake: the app blocks waiting for remote config changes, and the test blocks waiting for traces. Under CI load, the trace writer's first flush (including feature discovery) can exceed 10s. When the app's timeout fires first, it crashes the process, killing the daemon trace-writer thread before it delivers traces — so traceCount stays at 0. Aligning both timeouts to 30s ensures the test's polling condition is always the binding constraint, preventing the app from independently crashing. Observed across all JDK versions (8, 11, 17, 21, 25) — 17 failures in the last 30 days. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Co-authored-by: devflow.devflow-routing-intake <devflow.devflow-routing-intake@kubernetes.us1.ddbuild.io>
1 parent b6e89cd commit 847896e

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

dd-smoke-tests/log-injection/src/main/java/datadog/smoketest/loginjection/BaseApplication.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
import java.util.function.Supplier;
1111

1212
public abstract class BaseApplication {
13-
public static final long TIMEOUT_IN_NANOS = TimeUnit.SECONDS.toNanos(10);
13+
public static final long TIMEOUT_IN_NANOS = TimeUnit.SECONDS.toNanos(30);
1414

1515
public abstract void doLog(String message);
1616

0 commit comments

Comments
 (0)