Skip to content

ci: give JBang the Java 21 the Camel launcher needs, and keep the integration tests on 17 - #3091

Merged
davsclaus merged 3 commits into
apache:mainfrom
davsclaus:ci-fix/integration-tests-jdk21
Oct 6, 2026
Merged

davsclaus merged 3 commits into
apache:mainfrom
davsclaus:ci-fix/integration-tests-jdk21

Conversation

@davsclaus

@davsclaus davsclaus commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Claude Code on behalf of Claus Ibsen.

The Integration Tests workflow sometimes fails in "Run Tests" before any test runs:

[jbang] [ERROR] No suitable JDK was found for requested version: 21

The step runs jbang camel@apache/camel. Camel's JBang launcher has declared //JAVA 21+ since CAMEL-23112. With only JDK 17 set up, JBang has to download a JDK 21, and that download fails now and then. It passed on #3089 and #3090 at 16:38, and failed on #3090 at 16:07 and on #3093 at 17:53 and again at 18:35.

The first version of this PR switched the workflow to Java 21. Then JiraIT failed twice in a row (3 of 3, "Action timeout ... Failed to receive message on endpoint: 'jiraServer.inbound'"), while every other integration test passed. JiraIT passes on 17.

So this sets up both JDKs, with 17 as the default (listed last) for the Maven build and the Citrus tests. JBang doesn't look at the JDKs setup-java installs: with both installed it still failed with the same error, although JAVA_HOME_21_X64 was set. So the step links that one into JBang with jbang jdk install 21 "$JAVA_HOME_21_X64", and JBang no longer downloads a JDK at all.

The integration tests on this PR are the check.

…her needs

The tests run jbang camel@apache/camel, whose launcher declares //JAVA 21+ since
CAMEL-23112. With JDK 17 set up, JBang 0.142.0 fails with "No suitable JDK was
found for requested version: 21". The Maven builds stay on 17: Camel is still
compiled for Java 17.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01STT6whBgK1AqsSsUKrnE8m
@davsclaus
davsclaus requested review from Croway and oscerd October 6, 2026 16:40
…s out on 21

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01STT6whBgK1AqsSsUKrnE8m
@davsclaus davsclaus changed the title ci: run the integration tests on Java 21, which the Camel JBang launcher needs ci: install Java 21 beside 17 for the integration tests, which the Camel JBang launcher needs Oct 6, 2026
…etup-java installs

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01STT6whBgK1AqsSsUKrnE8m
@davsclaus davsclaus changed the title ci: install Java 21 beside 17 for the integration tests, which the Camel JBang launcher needs ci: give JBang the Java 21 the Camel launcher needs, and keep the integration tests on 17 Oct 6, 2026
@davsclaus
davsclaus merged commit 9783319 into apache:main Oct 6, 2026
4 of 5 checks passed
@davsclaus
davsclaus deleted the ci-fix/integration-tests-jdk21 branch October 6, 2026 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant