Commit 7c8927f
Fix check raw file injection smoke test (#10920)
Fix flaky check raw file injection test in LogInjectionSmokeTest
Three bugs contributed to flakiness in CI on JDK 8 variants (zulu8,
semeru8) and under load:
1. Copy-paste bug in assertRawLogLinesWithInjection: the fallback
assertion for 32-bit trace IDs incorrectly referenced logLines[0]
(BEFORE FIRST SPAN) instead of logLines[4] (INSIDE THIRD SPAN) and
logLines[6] (AFTER FORTH SPAN). This meant the 32-bit trace ID
format was never actually validated for those two log lines.
2. BaseApplication.waitForCondition timeout was 10 seconds. On loaded
CI machines with JDK 8 JVMs, the RC config change propagation through
captureTraceConfig() could take several seconds, approaching the
limit and causing "Logs injection config was never updated" failures.
Increased to 30 seconds to give adequate headroom.
Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
Revert test timeout increase
Co-authored-by: devflow.devflow-routing-intake <devflow.devflow-routing-intake@kubernetes.us1.ddbuild.io>1 parent 1916c90 commit 7c8927f
File tree
1 file changed
+2
-2
lines changed- dd-smoke-tests/log-injection/src/test/groovy/datadog/smoketest
1 file changed
+2
-2
lines changedLines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
231 | 231 | | |
232 | 232 | | |
233 | 233 | | |
234 | | - | |
| 234 | + | |
235 | 235 | | |
236 | | - | |
| 236 | + | |
237 | 237 | | |
238 | 238 | | |
239 | 239 | | |
| |||
0 commit comments