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

kvserver: skip replicate queue tests under metamorphic builds #100785

Merged
merged 1 commit into from
Apr 14, 2023

Conversation

AlexTalks
Copy link
Contributor

This modifies TestReplicateQueueDecommissioningNonVoters to be skipped specifically under metamorphic builds because this may enable mux range feeds, which are not currently working in these test builds (#100783). Once this issue is resolved and the test can work successfully with mux range feeds, this check should be removed.

Fixes: #99207

Release note: None

@AlexTalks AlexTalks requested a review from a team as a code owner April 6, 2023 03:16
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@AlexTalks AlexTalks requested review from kvoli and miretskiy April 6, 2023 03:16
@AlexTalks AlexTalks added the backport-23.1.x Flags PRs that need to be backported to 23.1 label Apr 6, 2023
Copy link
Collaborator

@kvoli kvoli left a comment

Choose a reason for hiding this comment

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

:lgtm:

Do we need to skip any other replicate queue tests too? They would all be affected by this?

Reviewed 2 of 2 files at r1, all commit messages.
Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained (waiting on @miretskiy)

This modifies `TestReplicateQueueDecommissioningNonVoters` to be skipped
specifically under metamorphic builds because this may enable mux range
feeds, which are not currently working in these test builds (cockroachdb#100783).
Once this issue is resolved and the test can work successfully with mux
range feeds, this check should be removed.

Fixes: cockroachdb#99207

Release note: None
@AlexTalks AlexTalks force-pushed the fix_rq_test_mux_range_feed branch from a5d0fbc to 8481c26 Compare April 14, 2023 00:36
@AlexTalks
Copy link
Contributor Author

bors r+

@craig
Copy link
Contributor

craig bot commented Apr 14, 2023

Build failed:

@AlexTalks
Copy link
Contributor Author

bors r+

@craig
Copy link
Contributor

craig bot commented Apr 14, 2023

Build succeeded:

@craig craig bot merged commit d50149d into cockroachdb:master Apr 14, 2023
@AlexTalks AlexTalks deleted the fix_rq_test_mux_range_feed branch April 18, 2023 00:26
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

kv/kvserver: TestReplicateQueueDecommissioningNonVoters failed
3 participants