-
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
acceptance: TestDockerCSharp failed #58218
Comments
(acceptance).TestDockerCSharp failed on master@7f3d64138b2efe23e62161543ce2872c14173b85:
MoreParameters:
See this test on roachdash |
(acceptance).TestDockerCSharp failed on master@08c89a597a06520c30faf01965f9c74fe9b9854f:
MoreParameters:
See this test on roachdash |
(acceptance).TestDockerCSharp failed on master@b2ac447d23c97332b705e288c055d63119fb8d28:
MoreParameters:
See this test on roachdash |
(acceptance).TestDockerCSharp failed on master@15765c0fa9118885dda0bd2ad1384b8801c412c3:
MoreParameters:
See this test on roachdash |
(acceptance).TestDockerCSharp failed on master@3b9c6d2cc5de8603d0474b49bed5594fbf34d093:
MoreParameters:
See this test on roachdash |
Refs: cockroachdb#58218 Reason: flaky test Generated by bin/skip-test. Release justification: non-production code changes Release note: None
(acceptance).TestDockerCSharp failed on master@23c9843a4cc900be9ad9c931f86dbf06c620d28a:
MoreParameters:
See this test on roachdash |
58706: acceptance: skip TestDockerCSharp r=adityamaru a=nkodali Refs: #58218 Reason: flaky test Generated by bin/skip-test. Release justification: non-production code changes Release note: None Co-authored-by: Namrata Kodali <[email protected]>
@spaskob What prompted you adding this issue to the dev-inf board? |
The docker part I guess. Feel free to re-assign - there was not a clear
other owner to me.
…On Tue, Feb 16, 2021 at 3:03 PM James H. Linder ***@***.***> wrote:
@spaskob <https://github.com/spaskob> What prompted you adding this issue
to the dev-inf board?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#58218 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AANL633NHGPUKYMEJCVCRYDS7LFRHANCNFSM4VGISMZQ>
.
|
Looks like a dupe of #58955. |
86815: insights: report high contention time r=matthewtodd a=matthewtodd This change aims to align with the logic for selecting transactions with "high contention time"[^1] in the transaction insights UI, but in practice, it's a best effort: Because of the sampling nature of tracing and because a transaction with "high" contention may be composed of many statements, each with only a little contention, there's not a guarantee that a highly contented transaction will be composed of highly contended statements. But, on balance, the best effort is still valuable, and we will call out actionable statements with high contention. [^1]: Note that the protobuf enum name change is safe here as we haven't yet deployed this code. Release justification: Category 2: Bug fixes and low-risk updates to new functionality. Release note: None 86846: storage: additional `TestMVCCHistories` cases for scan/get r=jbowens a=erikgrinaker Touches #86655. Release justification: non-production code changes Release note: None 86862: acceptance: fix spurious docker test failure `unexpected extra event &{0 die} (after [])` r=rickystewart,renatolabs a=knz Needed for #86049. The root cause of the issue is that the docker container for the DB nodes were crashing, because the test framework **was incorrectly deleting the data/log directory before shutting down the nodes**. However, this root cause was mostly hidden because of another mistake: the assertion that checks the health of the nodes was not doing its job properly in most cases. So it believed the nodes were healthy when in fact they were not. This PR fixes both problems. Informs #58955. Informs #58951. Informs #58218. Release justification: non-production code changes 86865: ttl: remove unused const expiredRowsPerRange r=knz a=ecwall fixes #86838 Release justification: Linter fix. Release note: None Co-authored-by: Matthew Todd <[email protected]> Co-authored-by: Erik Grinaker <[email protected]> Co-authored-by: Raphael 'kena' Poss <[email protected]> Co-authored-by: Evan Wall <[email protected]>
(acceptance).TestDockerCSharp failed on master@2708ab530553ea1096c79af63ed823199b7a5c8e:
More
Parameters:
See this test on roachdash
powered by pkg/cmd/internal/issues
The text was updated successfully, but these errors were encountered: