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

dev: testing under stress is broken #76385

Closed
irfansharif opened this issue Feb 10, 2022 · 0 comments · Fixed by #76390
Closed

dev: testing under stress is broken #76385

irfansharif opened this issue Feb 10, 2022 · 0 comments · Fixed by #76390
Assignees
@rickystewart
Labels
A-build-system C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.

Comments

@irfansharif
Copy link
Contributor

irfansharif commented Feb 10, 2022 •
edited
Loading

Is your feature request related to a problem? Please describe.

$ dev test pkg/spanconfig/spanconfigkvsubscriber -f=TestSpanConfigDecoder --stress --timeout 10s
[...]
146 runs completed, 0 failures, over 10s
ERROR: exit status 1
exit status 1
FAIL

Getting rid of the -shardable-artifacts/merge-test-xmls business we introduced in fff8684 from the expanded bazel command seems to fix it. Is there a bug here?

Epic CRDB-8036
@irfansharif irfansharif added C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. A-build-system labels Feb 10, 2022
@rickystewart rickystewart mentioned this issue Feb 10, 2022
Merged
craig bot pushed a commit that referenced this issue Feb 11, 2022
@rickystewart
Merge #76390
ce55af9 
76390: dev: disable `doctor` check for `merge-test-xmls` r=irfansharif a=rickystewart

Closes #76385.

Release note: None

Co-authored-by: Ricky Stewart <[email protected]>
@craig craig bot closed this as completed in 8b04772 Feb 11, 2022
RajivTS pushed a commit to RajivTS/cockroach that referenced this issue Mar 6, 2022
@rickystewart @RajivTS
dev: disable doctor check for merge-test-xmls
57738c0 
Closes cockroachdb#76385.

Release note: None
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rickystewart rickystewart

Labels
A-build-system C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

dev: disable doctor check for merge-test-xmls rickystewart/cockroach
2 participants
@rickystewart @irfansharif