-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
sql: disable span-config on flakey 5node tests #75281
Conversation
82a23e3
to
af7ceb9
Compare
@@ -1,4 +1,5 @@ | |||
# LogicTest: 5node-default-configs !5node-metadata | |||
# cluster-opt: disable-span-configs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we want to file an issue so that this can be removed when fixed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 #75282
af7ceb9
to
8271929
Compare
Temporary fix for cockroachdb#72802 Due to cockroachdb#73876 these tests have become flakey. With the disable-span-configs option 70 runs of make stress on the opt logic tests pass. Release note: None
8271929
to
422e00e
Compare
bors r+ |
Build failed (retrying...): |
This PR was included in a batch that was canceled, it will be automatically retried |
Build failed (retrying...): |
Build failed (retrying...): |
Build succeeded: |
We opted this test out of using the span configs infra in cockroachdb#75281 after observing a CI flake. Following the analysis in the last commit, it may have been due to stale distsender caches affecting the physical plan generated. This fix is speculative because I was unable to repro the original flake under stress. Attempt (successful after 1000s of runs on GCE worker over 20+ minutes): dev test pkg/sql/logictest \ -f 'TestLogic/^5node-disk$/distsql_enum' \ --show-logs -v --stress --stress-args '-p 4' -- --test_arg -show-sql Release note: None
Temporary fix for #72802 and 5node/distsql_enum CI failures.
Due to #73876 these tests have become flakey.
With the disable-span-configs option 70 runs of make stress on the opt logic tests pass.
Release note: None