Skip to content

roachtest: skip running backup schedule on c2c source#135071

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
msbutler:butler-deflake-c2c-tpcc
Nov 13, 2024
Merged

roachtest: skip running backup schedule on c2c source#135071
craig[bot] merged 1 commit intocockroachdb:masterfrom
msbutler:butler-deflake-c2c-tpcc

Conversation

@msbutler
Copy link
Copy Markdown
Collaborator

The c2c perf tests histoically have not run a backup schedule on the source. A recent refactor (#131940), lead to backups running on the source. During full backup execution, cpu utilization exceeded 90%, leading to overload and replication lag higher than 2 minutes. This patch removes the backup schedule.

At some point, we should run the c2c tests with a backup schedule, but that would require more hardware or more ac pacing work to stabilize the test. These are not priorities now.

Fixes #133233

Release note: none

The c2c perf tests histoically have not run a backup schedule on the source. A
recent refactor (cockroachdb#131940), lead to backups running on the source. During full
backup execution, cpu utilization exceeded 90%, leading to overload and
replication lag higher than 2 minutes. This patch removes the backup schedule.

At some point, we should run the c2c tests with a backup schedule, but that
would require more hardware or more ac pacing work to stabilize the test. These
are not priorities now.

Fixes cockroachdb#133233

Release note: none
@msbutler msbutler requested a review from kev-cao November 13, 2024 13:38
@msbutler msbutler self-assigned this Nov 13, 2024
@msbutler msbutler requested a review from a team as a code owner November 13, 2024 13:38
@msbutler msbutler requested review from DarrylWong and vidit-bhat and removed request for a team November 13, 2024 13:38
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@msbutler
Copy link
Copy Markdown
Collaborator Author

TFTR!

bors r=dt

@msbutler
Copy link
Copy Markdown
Collaborator Author

TFTR

bors r=dt

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Nov 13, 2024

Already running a review

@msbutler msbutler added the backport-24.3.x Flags PRs that need to be backported to 24.3 label Nov 13, 2024
@craig
Copy link
Copy Markdown
Contributor

craig bot commented Nov 13, 2024

@craig craig bot merged commit 615462e into cockroachdb:master Nov 13, 2024
@blathers-crl
Copy link
Copy Markdown

blathers-crl bot commented Nov 13, 2024

Based on the specified backports for this PR, I applied new labels to the following linked issue(s). Please adjust the labels as needed to match the branches actually affected by the issue(s), including adding any known older branches.


Issue #133233: branch-release-24.3.


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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-24.3.x Flags PRs that need to be backported to 24.3 T-disaster-recovery

Projects

None yet

Development

Successfully merging this pull request may close these issues.

roachtest: c2c/tpcc/warehouses=1000/duration=60/cutover=30 failed

3 participants