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.

Sign up for GitHub

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

Jump to bottom

logic: TestLogic_testserver_upgrade_node flaked #94956

Closed
msbutler opened this issue Jan 9, 2023 · 0 comments · Fixed by #95211
Closed

logic: TestLogic_testserver_upgrade_node flaked #94956

msbutler opened this issue Jan 9, 2023 · 0 comments · Fixed by #95211
Assignees
@rafiss
Labels
C-test-failure Broken test (automatically or manually discovered). skipped-test T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions)

Comments

@msbutler
Copy link
Collaborator

msbutler commented Jan 9, 2023 •
edited by cockroach-jira-scripts
Loading

See flake here. Going to skip.

Jira issue: CRDB-23243
@msbutler msbutler added C-test-failure Broken test (automatically or manually discovered). T-server-and-security DB Server & Security labels Jan 9, 2023
msbutler added a commit to msbutler/cockroach that referenced this issue Jan 9, 2023
@msbutler
kv/server: skip a bunch of flakey tests
ff6459e 
TestStoreCapacityAfterSplit, TestMergeQueueDoesNotInterruptReplicationChange,
testserver_test_22.1_22.2, testserve_upgrade_node.

Informs: cockroachdb#94871, cockroachdb#94951, cockroachdb#92677 cockroachdb#94956

Release note: None
@msbutler msbutler mentioned this issue Jan 9, 2023
Merged
craig bot pushed a commit that referenced this issue Jan 9, 2023
@msbutler
Merge #94954
1738c19 
94954: kv/server: skip a bunch of flakey tests r=adityamaru a=msbutler

TestStoreCapacityAfterSplit, TestMergeQueueDoesNotInterruptReplicationChange,
testserver_test_22.1_22.2, testserver_upgrade_node.

Informs: #94871, #94951, #92677, #94956

Release note: None

Co-authored-by: Michael Butler <[email protected]>
@rafiss rafiss self-assigned this Jan 11, 2023
@rafiss rafiss removed the T-server-and-security DB Server & Security label Jan 11, 2023
@blathers-crl blathers-crl bot added the T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions) label Jan 11, 2023
This was referenced Jan 13, 2023
Merged
Merged
@craig craig bot closed this as completed in f273aae Jan 14, 2023
@rafiss rafiss mentioned this issue Mar 13, 2023
Merged
craig bot pushed a commit that referenced this issue Mar 13, 2023
@rafiss @mgartner
Merge #98528 #98533
2ac6e51 
98528: logictest: use a retry loop when starting testserver r=rafiss a=rafiss

A similar change was already made for upgrading the node.

informs #94956

Release note: None

98533: ccl/multiregion: skip TestMrSystemDatabase r=mgartner a=mgartner

This test is flaky. See #98039.

Epic: None

Release note: None

Co-authored-by: Rafi Shamim <[email protected]>
Co-authored-by: Marcus Gartner <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rafiss rafiss

Labels
C-test-failure Broken test (automatically or manually discovered). skipped-test T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions)
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

logictest: increase timeout for listenurl file rafiss/cockroach
2 participants
@rafiss @msbutler