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

releasePlan: Switch jdk22 -> 22u and set tag suitable for dryrun #50

Merged
merged 1 commit into from
Apr 8, 2024

Conversation

sxa
Copy link
Member

@sxa sxa commented Apr 8, 2024

The latest STS releases do not have regular tags, therefore the dryrun build cannot be performed on this release as

scmReferenceLatest=$(git ls-remote --tags "${ADOPTIUM_REPO}" | grep "\^{}" | grep "${gaTagCommitSHA}" | tr -s "\t " " " | cut -d" " -f2 | sed "s,refs/tags/,," | sed "s,\^{},," | grep -v "\-ga" | sort -V -r | head -1)
expects there to be a "real" _adopt tag matching the -dryrun-ga one.

I am making this update with the intention of creating and pushing a jdk-22.0.0+0 which points to the same commit as jdk-22.0.0-dryrun-ga and will therefore allow the dryrun to progress. I'm choosing 22.0.0+0 as it will not interfere with (1) The current jdk-22+xx tags or (2) Future jdk-22.0.1+xx tags.

This will need to be switched over to 22.0.1 once the dryrun is complete and we prep for the GA.

Signed-off-by: Stewart X Addison <sxa@redhat.com>
Copy link
Contributor

@andrew-m-leonard andrew-m-leonard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Contributor

@steelhead31 steelhead31 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@sxa sxa merged commit 1193355 into master Apr 8, 2024
3 checks passed
@sxa sxa deleted the releaseplan-20.0.0 branch April 8, 2024 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants