Skip to content

roachtest: add back backups to disk-stalled tests#99871

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
nicktrav:nickt.roachtest-disk-stall-backups
Mar 30, 2023
Merged

roachtest: add back backups to disk-stalled tests#99871
craig[bot] merged 1 commit intocockroachdb:masterfrom
nicktrav:nickt.roachtest-disk-stall-backups

Conversation

@nicktrav
Copy link
Copy Markdown
Collaborator

Backups were inadvertently disabled in #99747 (to simplify testing). As there is no good reason to run without them, re-enable the backups during the test.

Release note: None.

Epic: CRDB-20293

Backups were inadvertently disabled in cockroachdb#99747 (to simplify testing). As
there is no good reason to run without them, re-enable the backups
during the test.

Release note: None.
@nicktrav nicktrav added the backport-23.1.x PAST MAINTENANCE SUPPORT: 23.1 patch releases via ER request only label Mar 28, 2023
@nicktrav nicktrav requested review from a team and jbowens March 28, 2023 22:01
@nicktrav nicktrav requested a review from a team as a code owner March 28, 2023 22:01
@nicktrav nicktrav requested review from smg260 and srosenberg and removed request for a team March 28, 2023 22:01
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

Copy link
Copy Markdown
Contributor

@jbowens jbowens 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 @smg260 and @srosenberg)

@nicktrav
Copy link
Copy Markdown
Collaborator Author

TFTR!

bors r=jbowens

@nicktrav
Copy link
Copy Markdown
Collaborator Author

bors retry

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Mar 29, 2023

Already running a review

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Mar 30, 2023

Build succeeded:

@craig craig bot merged commit 73f1291 into cockroachdb:master Mar 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-23.1.x PAST MAINTENANCE SUPPORT: 23.1 patch releases via ER request only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants