Skip to content

roachtest: use from version for workload#147377

Merged
craig[bot] merged 2 commits intocockroachdb:masterfrom
herkolategan:hbl/roachtest-rebalance-workload
May 28, 2025
Merged

roachtest: use from version for workload#147377
craig[bot] merged 2 commits intocockroachdb:masterfrom
herkolategan:hbl/roachtest-rebalance-workload

Conversation

@herkolategan
Copy link
Copy Markdown
Collaborator

@herkolategan herkolategan commented May 27, 2025

Previously, rebalance/by-load/leases/mixed-version used the master workload binary. This caused compatibility issue with mixed version tests. This change uploads the from version workload binary to the cluster, before running the workload.

Fixes: #147250
Informs: #147374

Epic: None
Release note: None

@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@herkolategan herkolategan force-pushed the hbl/roachtest-rebalance-workload branch 2 times, most recently from 0ec70cb to 50316c6 Compare May 28, 2025 07:01
@herkolategan
Copy link
Copy Markdown
Collaborator Author

herkolategan commented May 28, 2025

TeamCity run with the same seed (-7112154562575888906) that failed, previously: https://teamcity.cockroachdb.com/buildConfiguration/Cockroach_Nightlies_RoachtestNightlyGceBazel/19796609

Copy link
Copy Markdown
Collaborator

@stevendanna stevendanna left a comment

Choose a reason for hiding this comment

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

Awesome, thanks for tackling this one!

Previously, `rebalance/by-load/leases/mixed-version` used the master workload
binary. This caused compatibility issue with mixed version tests. This change
uploads the `from version` workload binary to the cluster, before running the
workload.

Fixes: cockroachdb#147250
Informs: cockroachdb#147374

Epic: None
Release note: None
@herkolategan herkolategan force-pushed the hbl/roachtest-rebalance-workload branch from f3e184a to fdb6651 Compare May 28, 2025 08:58
@herkolategan
Copy link
Copy Markdown
Collaborator Author

TFTR!

bors r=stevendanna

@craig
Copy link
Copy Markdown
Contributor

craig bot commented May 28, 2025

@craig craig bot merged commit 4eb8e22 into cockroachdb:master May 28, 2025
21 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

roachtest: rebalance/by-load/leases/mixed-version failed [unrecognized configuration parameter "autocommit_before_ddl"]

3 participants