-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
clusterversion: mint 25.1 #140412
clusterversion: mint 25.1 #140412
Conversation
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. |
5df5f83
to
1c39c18
Compare
[updated Feb 6, 1:15p PST]
resolved / no longer errors:
|
test failure 1 [of 4] is because
|
UPDATE: resolved ✅
|
UPDATE: resolved ✅
|
@dt @RaduBerinde - I could use your help figuring out why these version-related failures are happening, when I try to mint 25.1 - thank you! Tests are in these comments: |
UPDATE: resolved ✅
|
The
|
140479: admission: fix shutdown logic for store grant coordinator r=RaduBerinde a=aadityasondhi We were incorrectly waiting on the ticker outside of the select statement, which would inherently cause a delay in case of the stopping sequence being initiated. Informs #140454. Release note: None 140486: quantize: optimize single vector quantized sets r=andy-kimball a=andy-kimball Allocate array memory in-line for quantized sets that contain only a single vector. Epic: CRDB-42943 Release note: None 140489: roachpb: add pretty-printing for three remaining value tags r=rickystewart a=yuzefovich TIMETZ, GEO, and BOX2D value tags were missing support for pretty-printing. Epic: None Release note: None 140498: server: update `prev()` logic to account for more releases per year r=RaduBerinde a=celiala This fix is needed to unblock the minting of 25.1, see #140412 (comment). This PR adjusts `prev()` to account for the shift to more releases per year, where we followed this release cadence: - Before 2024, we shipped 2 releases a year. - In 2024, we shipped 3 releases. - And 2025 onwards, we will be shipping 4 releases a year. Release note: None Epic: REL-1751 Release justification: test-only fix, to unblock release-process. Co-authored-by: Aaditya Sondhi <[email protected]> Co-authored-by: Andrew Kimball <[email protected]> Co-authored-by: Yahor Yuzefovich <[email protected]> Co-authored-by: Celia La <[email protected]>
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.
Reviewable status:
complete! 1 of 0 LGTMs obtained
Release note: None Epic: REL-1751 Release justification: release-process change: mints 25.1.
I'm going to skip this failing test:
|
This PR:
checklist for next time:
SystemDatabaseSchemaBootstrapVersion
toV25_1
./dev test pkg/sql/catalog/bootstrap --rewrite -f TestInitialValuesToString
Release note: None
Epic: REL-1751
Release justification: release-process change: mints 25.1.