Skip to content

kvserver: skip replicate queue tests under metamorphic builds#100785

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
AlexTalks:fix_rq_test_mux_range_feed
Apr 14, 2023
Merged

kvserver: skip replicate queue tests under metamorphic builds#100785
craig[bot] merged 1 commit intocockroachdb:masterfrom
AlexTalks:fix_rq_test_mux_range_feed

Conversation

@AlexTalks
Copy link
Copy Markdown
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
Copy Markdown
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 PAST MAINTENANCE SUPPORT: 23.1 patch releases via ER request only label Apr 6, 2023
Copy link
Copy Markdown
Contributor

@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
Copy Markdown
Contributor Author

bors r+

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Apr 14, 2023

Build failed:

@AlexTalks
Copy link
Copy Markdown
Contributor Author

bors r+

@craig
Copy link
Copy Markdown
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 PAST MAINTENANCE SUPPORT: 23.1 patch releases via ER request only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

kv/kvserver: TestReplicateQueueDecommissioningNonVoters failed

3 participants