Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

streamingccl: skip acceptance/c2c on remote cluster setup #99574

Merged
merged 1 commit into from
Mar 28, 2023

Conversation

msbutler
Copy link
Collaborator

acceptance/c2c currently fails when run on a remote cluster. This patch ensures the test gets skipped when run on a remote cluster. There's no need to run the test on a remote cluster because the other c2c roachtests provide sufficient coverage.

Fixes #99553

Release note: none

@msbutler msbutler self-assigned this Mar 25, 2023
@msbutler msbutler requested a review from a team as a code owner March 25, 2023 15:24
@msbutler msbutler requested review from herkolategan and srosenberg and removed request for a team March 25, 2023 15:24
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@msbutler msbutler changed the title streamingccl: allow acceptance/c2c to run on remote cluster streamingccl: skip acceptance/c2c on remote cluster setup Mar 25, 2023
@msbutler msbutler force-pushed the butler-fix-acceptance-flake branch from 54fbdc8 to c1e8067 Compare March 25, 2023 20:53
@msbutler msbutler added the backport-23.1.x Flags PRs that need to be backported to 23.1 label Mar 27, 2023
@msbutler
Copy link
Collaborator Author

TFTR!

bors r=stevendanna

@craig
Copy link
Contributor

craig bot commented Mar 27, 2023

Build failed:

@msbutler
Copy link
Collaborator Author

bors r=retry

@craig
Copy link
Contributor

craig bot commented Mar 27, 2023

Build failed:

@msbutler
Copy link
Collaborator Author

bors r=retry

@craig
Copy link
Contributor

craig bot commented Mar 27, 2023

Build failed (retrying...):

@craig
Copy link
Contributor

craig bot commented Mar 28, 2023

Build failed:

acceptance/c2c currently fails when run on a remote cluster. This patch ensures
the test gets skipped when run on a remote cluster. There's no need to run the
test on a remote cluster because the other c2c roachtests provide sufficient
coverage.

Fixes cockroachdb#99553

Release note: none
@msbutler msbutler force-pushed the butler-fix-acceptance-flake branch from c1e8067 to a8dd57c Compare March 28, 2023 11:03
@msbutler
Copy link
Collaborator Author

let's hope bors likes me this time.

bors r=stevendanna

@craig craig bot merged commit 0745cd4 into cockroachdb:master Mar 28, 2023
@craig
Copy link
Contributor

craig bot commented Mar 28, 2023

Build succeeded:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-23.1.x Flags PRs that need to be backported to 23.1 T-disaster-recovery
Projects
None yet
Development

Successfully merging this pull request may close these issues.

roachtest: acceptance/c2c failed
3 participants