Megha-Megha13 commented on PR #28838: URL: https://github.com/apache/camel/pull/28838#issuecomment-6641401478
> Thanks for activating these. I checked the ppc64le image: `icr.io/ppc64le-oss/docling-serve-ppc64le:1.35.0` is a linux/ppc64le image with `docling-serve run` as its command and port 5001, and the quay.io v1.38.2 image is amd64/arm64 only, so the override is needed. s390x stays skipped, which is fine. > > One question: `camel.failsafe.forkTimeout=1800` in the camel-docling pom applies on every architecture, so a hung IT on x86 now runs for 30 minutes instead of the parent's 10. Could it go in a profile activated only on ppc64le (`<activation><os><arch>ppc64le</arch></os></activation>`), like the os.arch checks you added for the process timeout and `DOCLING_SERVE_MAX_SYNC_WAIT`? > > The earlier conditional assertion in `DoclingInputSourceValidationTest` is gone from the diff, so @apupier's point looks addressed. Waiting for the Build and test run to finish. > > _Claude Code on behalf of Claus Ibsen (davsclaus). This review was generated by an AI agent and may contain inaccuracies; please verify all suggestions before applying._ Thanks for the review. I have addressed the suggestion by moving `camel.failsafe.forkTimeout=1800` into a Maven profile activated only on ppc64le. I have pushed the changes and verified that the complete test suite passes on ppc64le. I have also attached the latest successful test log for reference. -- 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]
