Skip to content

release: verify version.txt in Pick SHA#112687

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
rail:pick-sha-verify-version
Oct 19, 2023
Merged

release: verify version.txt in Pick SHA#112687
craig[bot] merged 1 commit intocockroachdb:masterfrom
rail:pick-sha-verify-version

Conversation

@rail
Copy link
Copy Markdown
Member

@rail rail commented Oct 19, 2023

Previously, we did not verify the contents of the version.txt file as a part of the Pick SHA step. In case the guessed version doesn't match the contents of the file, we have to delay the release by many hours to allow CI automation work.

This PR adds a check in the Pick SHA step to verify the version.

Fixes: RE-438
Release note: None
Release justification: release automation changes

@rail rail added C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) A-release T-release Release Engineering & Automation Team backport-23.1.x PAST MAINTENANCE SUPPORT: 23.1 patch releases via ER request only backport-23.2.x PAST MAINTENANCE SUPPORT: 23.2 patch releases via ER request only labels Oct 19, 2023
@rail rail requested a review from celiala October 19, 2023 14:13
@rail rail requested a review from a team as a code owner October 19, 2023 14:13
@rail rail self-assigned this Oct 19, 2023
@blathers-crl
Copy link
Copy Markdown

blathers-crl bot commented Oct 19, 2023

It looks like your PR touches production code but doesn't add or edit any test code. Did you consider adding tests to your PR?

🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf.

@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@rail rail force-pushed the pick-sha-verify-version branch from 98d00fa to 3e1ad2d Compare October 19, 2023 14:30
Previously, we did not verify the contents of the version.txt file as a
part of the Pick SHA step. In case the guessed version doesn't match the
contents of the file, we have to delay the release by many hours to
allow CI automation work.

This PR adds a check in the Pick SHA step to verify the version.

Fixes: RE-438
Release note: None
@rail rail force-pushed the pick-sha-verify-version branch from 3e1ad2d to 208c24b Compare October 19, 2023 14:42
Copy link
Copy Markdown
Collaborator

@celiala celiala left a comment

Choose a reason for hiding this comment

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

Thank you! LGTM!

Copy link
Copy Markdown
Collaborator

@celiala celiala left a comment

Choose a reason for hiding this comment

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

Thank you! LGTM!

@rail
Copy link
Copy Markdown
Member Author

rail commented Oct 19, 2023

bors r=celiala

@craig craig bot merged commit e4a11bc into cockroachdb:master Oct 19, 2023
@craig
Copy link
Copy Markdown
Contributor

craig bot commented Oct 19, 2023

Build succeeded:

@rickystewart
Copy link
Copy Markdown
Collaborator

Nice.

@rail rail deleted the pick-sha-verify-version branch March 14, 2025 19:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-release backport-23.1.x PAST MAINTENANCE SUPPORT: 23.1 patch releases via ER request only backport-23.2.x PAST MAINTENANCE SUPPORT: 23.2 patch releases via ER request only C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) T-release Release Engineering & Automation Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants