Skip to content

roachtest: update generate-fixtures to use clusterupgrade functions#131467

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
renatolabs:rc/roachtest-generate-fixtures-remove-old-api
Sep 27, 2024
Merged

roachtest: update generate-fixtures to use clusterupgrade functions#131467
craig[bot] merged 1 commit intocockroachdb:masterfrom
renatolabs:rc/roachtest-generate-fixtures-remove-old-api

Conversation

@renatolabs
Copy link
Copy Markdown

This commit updates the generate-fixtures roachtest, used to automate the generation of fixtures every release, so that it uses plain clusterupgrade functions instead of the newUpgradeTest API.

The old "step" functions only provided thin wrappers around the functionality of clusterupgrade. With this change, we are now able to completely remove the newUpgradeTest API which has been deprecated for a while in favor of the mixedversion framework.

Fixes: #110528

Release note: None

This commit updates the `generate-fixtures` roachtest, used to
automate the generation of fixtures every release, so that it uses
plain `clusterupgrade` functions instead of the `newUpgradeTest` API.

The old "step" functions only provided thin wrappers around the
functionality of `clusterupgrade`. With this change, we are now able
to completely remove the `newUpgradeTest` API which has been
deprecated for a while in favor of the `mixedversion` framework.

Fixes: cockroachdb#110528

Release note: None
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@renatolabs renatolabs marked this pull request as ready for review September 26, 2024 23:06
@renatolabs renatolabs requested a review from a team as a code owner September 26, 2024 23:06
@renatolabs renatolabs requested review from DarrylWong and herkolategan and removed request for a team September 26, 2024 23:06
@renatolabs
Copy link
Copy Markdown
Author

Ran generate-fixtures on gceworker on this branch and it works fine.

@srosenberg srosenberg self-requested a review September 27, 2024 01:26
Copy link
Copy Markdown
Member

@srosenberg srosenberg left a comment

Choose a reason for hiding this comment

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

Sweet!

@renatolabs
Copy link
Copy Markdown
Author

TFTR!

bors r=srosenberg

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Sep 27, 2024

@craig craig bot merged commit 67dc7a1 into cockroachdb:master Sep 27, 2024
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.

roachtest: port remaining mixed-version roachtests to the new framework

3 participants