jamesnetherton commented on PR #9118: URL: https://github.com/apache/camel-quarkus/pull/9118#issuecomment-5566448412
Thinking it might be best to leave this upgrade until after 3.40. Analysis from Claude: > Root cause > > The Dependabot bump from 3.5.6 → 3.6.0 pulls in SUREFIRE-823: "Decouple -DskipTests from Failsafe plugin". Before 3.6.0, -DskipTests skipped both Surefire and Failsafe. As of 3.6.0 it skips only Surefire — Failsafe now needs -DskipITs (or maven.test.skip). > > In this repo maven-failsafe-plugin shares the same version property (pom.xml:502), so the bump changed both. > > The initial-mvn-install job runs: > mvn ... clean install -DskipTests -Dquarkus.build.skip -Pformat > - -DskipTests previously suppressed all ITs. Now it no longer suppresses Failsafe. > - perf-regression's full profile (active because -Dquickly is not set) binds Failsafe integration-test/verify > unconditionally, so PerfRegressionIT now runs. > - But the same job passes -Dquarkus.build.skip → "Skipping Quarkus build" → quarkus-run.jar is never produced → > Error: Unable to access jarfile target/quarkus-app/quarkus-run.jar → test fails. > > perf-regression is the only affected module (other modules bind Failsafe only under -Dnative). The IT is meant to run in > the separate cd tooling && mvn verify job, which builds the Quarkus app normally. > > The same broken -DskipTests -Dquarkus.build.skip install pattern also appears in camel-master-cron.yaml, quarkus-> > master-cron.yaml, quarkus-lts-ci-build.yaml, and alternate-jdk-build.yaml — so all five would fail the same way. -- 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]
