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.

Sign up for GitHub

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

Jump to bottom

DLPX-88424 Update jdk artifact versions and respective sha values in host-jdks #27

Merged
merged 2 commits into from
Oct 27, 2023
Merged

DLPX-88424 Update jdk artifact versions and respective sha values in host-jdks #27

merged 2 commits into from
Oct 27, 2023

Conversation

lschetanrao
Copy link
Contributor

@lschetanrao lschetanrao commented Oct 17, 2023 •
edited
Loading

Description

Update the toolkit's JDKs to the 8u382b05 version.

Code changes

  • Updated the java versions and SHA values. (take sha values from delphix-java-packages repository)

Testing

ab-pre-push : http://selfservice.jenkins.delphix.com/job/github/job/delphix/job/host-jdks/job/appliance-build-orchestrator/job/pre-push/25/ ✅

Manual Testing

  • Ran ab-pre-push and cloned image.
  • Created environments in the latest image for AIX, HPUX, RHEL, solaris_sparcv9, solaris_x64, windows and checked jdk version of the toolkit - All are the latest as expected.
  • Tested SUSE 15 environment as well.

Bonus

Corresponding review:

@lschetanrao lschetanrao force-pushed the dlpx/pr/lschetanrao/aa4d47e5-f902-4b28-a68b-d0a79ef5d873 branch from 8b22ed6 to 2c37736 Compare October 17, 2023 09:27
@lschetanrao lschetanrao marked this pull request as ready for review October 26, 2023 10:50
@lschetanrao lschetanrao merged commit 4578f82 into develop Oct 27, 2023
3 of 7 checks passed
@lschetanrao lschetanrao deleted the dlpx/pr/lschetanrao/aa4d47e5-f902-4b28-a68b-d0a79ef5d873 branch October 27, 2023 06:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rasantel rasantel rasantel approved these changes

@nealquigley nealquigley nealquigley approved these changes

@VenkatanadhanG VenkatanadhanG VenkatanadhanG approved these changes

@rSohane rSohane rSohane approved these changes

@ShibasishDelphix ShibasishDelphix Awaiting requested review from ShibasishDelphix

@dbshah12 dbshah12 Awaiting requested review from dbshah12

@sebroy sebroy Awaiting requested review from sebroy

@seanatdelphix seanatdelphix Awaiting requested review from seanatdelphix

Assignees

@lschetanrao lschetanrao

Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@lschetanrao @rasantel @nealquigley @VenkatanadhanG @rSohane