gnodet-bot commented on code in PR #28838:
URL: https://github.com/apache/camel/pull/28838#discussion_r4472965163
##########
test-infra/camel-test-infra-docling/src/main/java/org/apache/camel/test/infra/docling/services/DoclingLocalContainerInfraService.java:
##########
@@ -68,6 +68,9 @@ public TestInfraDoclingContainer(boolean fixedPort) {
waitingFor(Wait.forListeningPorts(DOCLING_PORT))
.withStartupTimeout(Duration.ofMinutes(3L));
+ // Match the 300-second test timeout; Docling defaults to 120
seconds.
Review Comment:
💡 The comment says **"300-second test timeout"** but on the very next line
the value is conditionally **600** on ppc64le. Consider updating the comment to
reflect both cases, e.g.:
> `// Match the test timeout (300 s on x86_64, 600 s on ppc64le); Docling
defaults to 120 seconds.`
--
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]
