Skip to content

upgrades: use v22_2 bootstrap schema with role_members test#98613

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
jeffswenson:jeffswenson-fix-role-members-test
Mar 15, 2023
Merged

upgrades: use v22_2 bootstrap schema with role_members test#98613
craig[bot] merged 1 commit intocockroachdb:masterfrom
jeffswenson:jeffswenson-fix-role-members-test

Conversation

@jeffswenson
Copy link
Copy Markdown
Collaborator

Previously the role members test used the v23_1 schema with the V23_1RoleMembersTableHasIDColumns version gate. This caused problems for PR #98446 because the lease schema does not match the lease version gates.

Now, the test uses the BootstrapVersionKeyOverride to bootstrap with the V22_2 schema. Setting the cluster version walks the cluster through all of the migrations including the role members migration.

Part of #98446

Release Note: none

Previously the role members test used the v23_1 schema with the
V23_1RoleMembersTableHasIDColumns version gate. This caused problems for
PR cockroachdb#98446 because the lease schema does not match the lease version
gates.

Now, the test uses the BootstrapVersionKeyOverride to bootstrap with the
V22_2 schema. Setting the cluster version walks the cluster through all
of the migrations including the role members migration.

Part of cockroachdb#98446

Release Note: none
@jeffswenson jeffswenson requested review from a team and ajwerner March 14, 2023 19:25
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

Copy link
Copy Markdown
Contributor

@ajwerner ajwerner left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained (waiting on @jeffswenson)

@jeffswenson
Copy link
Copy Markdown
Collaborator Author

bors r+

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Mar 14, 2023

Build failed (retrying...):

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Mar 14, 2023

Build failed:

@jeffswenson
Copy link
Copy Markdown
Collaborator Author

bors r+

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Mar 15, 2023

Build failed (retrying...):

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Mar 15, 2023

Build succeeded:

@craig craig bot merged commit f8ca1e6 into cockroachdb:master Mar 15, 2023
@jeffswenson jeffswenson deleted the jeffswenson-fix-role-members-test branch March 16, 2023 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants