-
-
Notifications
You must be signed in to change notification settings - Fork 243
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
When auto-downloading boot JDK, default to hotspot (Don't use temurin in API) #2853
Conversation
… in API) Signed-off-by: Stewart X Addison <sxa@redhat.com>
ping @AdamBrousseau I don't think this will negatively affect you (for production levels we should have them on the machine already, and the APIs won't download the latest Semeru anyway), but if you believe otherwise let me know. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is fine. I believe I use node variables for most/all bootjdks anyways. Once IBM API is available I will probably add a change here to support it for openj9 variant.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@sxa worth checking with Alibaba? |
Dragonwell is not available through the Adopt API so this will not affect them, but it's needed for Temurin JDK19 builds to succeed. |
NOTE: The Dragonwell boot JDK issue that lead to the previous two comments is referred to in #2353 |
… in API) (adoptium#2853) Signed-off-by: Stewart X Addison <sxa@redhat.com>
I don't believe this will adversely affect anyone any more, so hard coding this in light of the changes made in #2818 so that the API call does not try to use
temurin
Alternate solution would be to use a bash specific construct of
${VARIANT/temurin/hotspot}
(Causes JDK19 to fail to build as it won't be able to download a boot JDK and we don't install that one via the playbooks yet as it's not a GA level)
Signed-off-by: Stewart X Addison sxa@redhat.com