Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ci] Skip JDK install if near match is found in $(JI_JAVA_HOME) #9585

Merged
merged 3 commits into from
Dec 4, 2024

Conversation

pjcollins
Copy link
Member

The JDK install step has been updated to optionally skip installation
if a close enough match is found in the $(JI_JAVA_HOME) directory. This
should allow us to skip JDK provisioning on our test lanes.

The JDK install step has been updated to optionally skip installation
if a close enough match is found in the $(JI_JAVA_HOME) directory. This
should allow us to skip JDK provisioning on our test lanes.
@pjcollins pjcollins marked this pull request as ready for review December 3, 2024 23:33
@pjcollins pjcollins requested a review from grendello as a code owner December 3, 2024 23:33
@pjcollins
Copy link
Member Author

Looking at a few test lanes this seems to be working as planned, assuming the rest of CI turns out well:

Microsoft OpenJDK with version 17.0.13 already installed in: /Users/runner/hostedtoolcache/Java_Temurin-Hotspot_jdk/17.0.13-11/x64/Contents/Home/

Microsoft OpenJDK with version 17.0.13 already installed in: C:\hostedtoolcache\windows\Java_Temurin-Hotspot_jdk\17.0.13-11\x64

@pjcollins pjcollins merged commit bf1a482 into main Dec 4, 2024
58 checks passed
@pjcollins pjcollins deleted the dev/pjc/skip-jdk-ci branch December 4, 2024 16:50
grendello added a commit that referenced this pull request Dec 5, 2024
* main:
  [native] More C++ tweaks and changes (#9478)
  Add android-platform-support to .gitignore (#9590)
  [ci] Skip JDK install if near match is found in $(JI_JAVA_HOME) (#9585)
  Localized file check-in by OneLocBuild Task: Build definition ID 17928: Build ID 10653360 (#9587)
  Revert "Try dependabot max_length param (#9529)"
  Try dependabot max_length param (#9529)
  LEGO: Pull request (#9575)
  Bump to DevDiv/android-platform-support@52dd010a (#9553)
  Localized file check-in by OneLocBuild Task (#9574)
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.

3 participants