Skip to content

Issue #17701: fix Cirrus CI Windows JDK21 installation by switching to Temurin#17702

Merged
romani merged 1 commit into
checkstyle:masterfrom
Brijeshthummar02:cirrus-ci-jdk21
Aug 28, 2025
Merged

Issue #17701: fix Cirrus CI Windows JDK21 installation by switching to Temurin#17702
romani merged 1 commit into
checkstyle:masterfrom
Brijeshthummar02:cirrus-ci-jdk21

Conversation

@Brijeshthummar02

@Brijeshthummar02 Brijeshthummar02 commented Aug 27, 2025

Copy link
Copy Markdown
Contributor

Fixes #17701

This PR fixes the Windows build on Cirrus CI, which was failing due to an unavailable JDK version.

  • Previously, .cirrus.yml tried to install OpenJDK 21.0.6 via Chocolatey:

    choco install -y --no-progress openjdk --version 21.0.6

    but Chocolatey's openjdk package only provides 21.0.0, so the job failed.

  • Updated .cirrus.yml to install Eclipse Temurin 21.0.6.7, which is available on Chocolatey.

  • Updated JAVA_HOME to point to the Temurin install path:

    C:\Program Files\Eclipse Adoptium\jdk-21

  • Added refreshenv to reload environment variables after the install step.

@Brijeshthummar02

Copy link
Copy Markdown
Contributor Author

@romani as you see on my code editor it is perfect TEMURIN_PATH: "Eclipse Adoptium\jdk-21"

image

But in PR how it showing red across letter 'J' don't know

image

@romani

romani commented Aug 27, 2025

Copy link
Copy Markdown
Member

Thanks a lot for quick fix.
Please make CI green.

If too much effort, we can use 21.0.0. exact latest jdk version is no critical to us.

@Brijeshthummar02

Copy link
Copy Markdown
Contributor Author

Thanks a lot for quick fix. Please make CI green.

If too much effort, we can use 21.0.0. exact latest jdk version is no critical to us.

i tried to fix current possible way if it still goes down, i will go by your way as said.

@Brijeshthummar02

Copy link
Copy Markdown
Contributor Author

@romani it should be fixed as said, all build were green and working with jdk21 been installed.
Minor CI for Gradle been failed earlier when i pushed.

@Brijeshthummar02

Copy link
Copy Markdown
Contributor Author

@romani All ci green, good to go?

@romani romani merged commit 1e22276 into checkstyle:master Aug 28, 2025
119 checks passed
@Brijeshthummar02 Brijeshthummar02 deleted the cirrus-ci-jdk21 branch August 28, 2025 16:47
@romani

romani commented Oct 1, 2025

Copy link
Copy Markdown
Member

@Brijeshthummar02 , please suggest what might be wrong in Cirrus ?
https://cirrus-ci.com/task/5358330002014208

java is not recognized

C:\Users\ContainerAdministrator\AppData\Local\Temp\cirrus-ci-build>call ant -version 
'"java.exe"' is not recognized as an internal or external command,
operable program or batch file.

@Brijeshthummar02

Copy link
Copy Markdown
Contributor Author

@Brijeshthummar02 , please suggest what might be wrong in Cirrus ? https://cirrus-ci.com/task/5358330002014208

java is not recognized

C:\Users\ContainerAdministrator\AppData\Local\Temp\cirrus-ci-build>call ant -version 
'"java.exe"' is not recognized as an internal or external command,
operable program or batch file.

Back than when we merged it was working, let me check what's the issue.

@Brijeshthummar02

Copy link
Copy Markdown
Contributor Author

@romani can we re-run complete ci/cd for last to PR where it got failed.

image

@romani

romani commented Oct 1, 2025

Copy link
Copy Markdown
Member

Cirrus does not run on PRs, it executes only after merge, only from code on master.
Last month we exceeded limit of usage, so it was disabled, so we didn't notice a moment of failure

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.

fix cirrus-ci failure to install jdk21

2 participants