Skip to content

Upgrade cimg/openjdk CI job containers when upgrading Java - #1258

Merged
jkschneider merged 1 commit into
java25/jaxb-api-for-importsfrom
java25/ci-container-image
Oct 3, 2026
Merged

jkschneider merged 1 commit into
java25/jaxb-api-for-importsfrom
java25/ci-container-image

Conversation

@jkschneider

Copy link
Copy Markdown
Member

CircleCI's docker: - image: cimg/openjdk:17.0 and a GitHub Actions container: cimg/openjdk:22.0 keep an older JDK that cannot compile or load the migrated classes. New UpgradeJavaCiContainerImage, added to UpgradeJavaVersion: in .github/workflows/*.yml and .circleci/config.yml, raise cimg/openjdk:<major>.<minor>[-variant] below the target to cimg/openjdk:<version>.0[-variant], leaving newer versions and other images alone.

Found by a team of coding agents reviewing a Moderne run of org.openrewrite.java.migrate.UpgradeToJava25 over eight open-source repositories (run 20261003103223-WtyA8). The issue is ci-container-java-version in the run's io.moderne.RecipeIssues data table, and the fix was discussed and reviewed on the run's message board. A row from the table, as a generalized example:

CI job containers retain an older Java version that cannot compile or load the migrated Java 25 classes. (rrd4j/rrd4j, .circleci/config.yml)

Before:

jobs:
  test:
    docker:
      - image: cimg/openjdk:17.0

What the recipe produced:

jobs:
  test:
    docker:
      - image: cimg/openjdk:17.0

Expected:

jobs:
  test:
    docker:
      - image: cimg/openjdk:25.0

Stacked on #1257 (9 of 9); merge that one first.

@jkschneider
jkschneider added this pull request to stack #1259 October 3, 2026 15:34
@jkschneider
jkschneider force-pushed the java25/ci-container-image branch from 7d0ddc3 to 7530506 Compare October 3, 2026 15:52
@jkschneider
jkschneider merged commit fd9c065 into main Oct 3, 2026
1 check passed
@jkschneider
jkschneider deleted the java25/ci-container-image branch October 3, 2026 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant