davsclaus opened a new pull request, #24774: URL: https://github.com/apache/camel/pull/24774
## Summary _Claude Code on behalf of davsclaus_ Fix flaky lumberjack tests caused by incomplete Netty resource cleanup between test runs: - **Await `shutdownGracefully()` in `LumberjackServer.stop()`**: the three Netty thread groups (`bossGroup`, `workerGroup`, `executorService`) were shut down fire-and-forget, so the server port could still be in use when the next `@Isolated` test tried to bind. Now awaits completion with `syncUninterruptibly()`. - **Await client channel close and event loop in `LumberjackUtil`**: `channel.close()` was fire-and-forget and `shutdownGracefully()` on the client event loop was not awaited, which could leave half-open connections that cause spurious ack count mismatches on the server side. - **Thread-safe `ErrorProcessor.count`**: the `int count` field in `LumberjackDisconnectionTest.ErrorProcessor` was read/written from Netty executor threads without synchronization. Replaced with `AtomicInteger`. - **Explicit Awaitility timeout**: the `Awaitility.await()` call in `LumberjackUtil.sendMessages()` relied on the 10-second default, which can be tight on slow CI nodes (especially for SSL tests). Now uses an explicit 30-second timeout. ## Test plan - [x] `mvn test` in `components/camel-lumberjack` — all 5 tests pass - [ ] CI validates no regressions across platforms 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
