This is an automated email from the ASF dual-hosted git repository.
gnodet pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git
The following commit(s) were added to refs/heads/main by this push:
new 5b3870618823 CAMEL-23427: Fix flaky SpanPropagationUpstreamTest in
camel-telemetry-dev
5b3870618823 is described below
commit 5b3870618823109a0657a9b3a8047e7f813a9b78
Author: Guillaume Nodet <[email protected]>
AuthorDate: Wed May 6 10:27:39 2026 +0200
CAMEL-23427: Fix flaky SpanPropagationUpstreamTest in camel-telemetry-dev
Change clearLogTraces() from @AfterEach to @BeforeEach to ensure the trace
log file is rolled over before each test runs, not after. With @AfterEach, the
rollover happened before CamelTestSupport.tearDown() stopped the context (JUnit
5 runs subclass @AfterEach before superclass), so any trace output during
context shutdown would be written to the new clean log file and picked up by
the next test as a stale trace.
---
.../org/apache/camel/telemetrydev/TelemetryDevTracerTestSupport.java | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
a/components/camel-telemetry-dev/src/test/java/org/apache/camel/telemetrydev/TelemetryDevTracerTestSupport.java
b/components/camel-telemetry-dev/src/test/java/org/apache/camel/telemetrydev/TelemetryDevTracerTestSupport.java
index 56f63d67d5c3..0b7f67e96f3f 100644
---
a/components/camel-telemetry-dev/src/test/java/org/apache/camel/telemetrydev/TelemetryDevTracerTestSupport.java
+++
b/components/camel-telemetry-dev/src/test/java/org/apache/camel/telemetrydev/TelemetryDevTracerTestSupport.java
@@ -31,7 +31,7 @@ import org.apache.camel.test.junit6.ExchangeTestSupport;
import org.apache.logging.log4j.LogManager;
import org.apache.logging.log4j.core.LoggerContext;
import org.apache.logging.log4j.core.appender.RollingFileAppender;
-import org.junit.jupiter.api.AfterEach;
+import org.junit.jupiter.api.BeforeEach;
public class TelemetryDevTracerTestSupport extends ExchangeTestSupport {
@@ -61,7 +61,7 @@ public class TelemetryDevTracerTestSupport extends
ExchangeTestSupport {
* This one is required to rollover the log traces database file and make
sure each test has its own
* set of fresh data.
*/
- @AfterEach
+ @BeforeEach
public synchronized void clearLogTraces() throws IOException {
final LoggerContext ctx = (LoggerContext) LogManager.getContext(false);
RollingFileAppender appender = (RollingFileAppender)
ctx.getConfiguration().getAppenders().get("file2");