-
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
pkg/ccl/serverccl/statusccl/statusccl_test: TestTenantStatusAPI failed #92382
Comments
pkg/ccl/serverccl/statusccl/statusccl_test.TestTenantStatusAPI failed with artifacts on release-22.2 @ ca673993473629a46370100108dd8c185a592e91:
Parameters: |
pkg/ccl/serverccl/statusccl/statusccl_test.TestTenantStatusAPI failed with artifacts on release-22.2 @ adf24a585553397a7ad8e4bb6da0609bb5d2e930:
Parameters: Same failure on other branches
|
pkg/ccl/serverccl/statusccl/statusccl_test.TestTenantStatusAPI failed with artifacts on release-22.2 @ 97d5489387634dfd25462c01cace49445d5698de:
Parameters: Same failure on other branches
|
pkg/ccl/serverccl/statusccl/statusccl_test.TestTenantStatusAPI failed with artifacts on release-22.2 @ ff69f981a02ea6906d4be138189c4531a6a420d1:
Parameters: Same failure on other branches
|
pkg/ccl/serverccl/statusccl/statusccl_test.TestTenantStatusAPI failed with artifacts on release-22.2 @ 8ff653b220128d32bbd7eff2717d6212b2b6ac12:
Parameters: Same failure on other branches
|
pkg/ccl/serverccl/statusccl/statusccl_test.TestTenantStatusAPI failed with artifacts on release-22.2 @ 07a53a36601e9ca5fcffcff55f69b43c6dfbf1c1:
Parameters: Same failure on other branches
|
pkg/ccl/serverccl/statusccl/statusccl_test.TestTenantStatusAPI failed with artifacts on release-22.2 @ cce52c9eab1210c6cd508f6b99b2bad0ba3e1e56:
Parameters: Same failure on other branches
|
pkg/ccl/serverccl/statusccl/statusccl_test.TestTenantStatusAPI failed with artifacts on release-22.2 @ cce0718a324e36d4144cc44a61186d26043b5601:
Parameters: Same failure on other branches
|
pkg/ccl/serverccl/statusccl/statusccl_test.TestTenantStatusAPI failed with artifacts on release-22.2 @ 82ac4cd732996b6e42870ce7e18306ecab214308:
Parameters: Same failure on other branches
|
pkg/ccl/serverccl/statusccl/statusccl_test.TestTenantStatusAPI failed with artifacts on release-22.2 @ 799d8dbba62fa9c942a780584c71f8ceab784381:
Parameters: Same failure on other branches
|
pkg/ccl/serverccl/statusccl/statusccl_test.TestTenantStatusAPI failed with artifacts on release-22.2 @ 92959b31f134f9d8a0a60c51276c82635a18c2a8:
Parameters: Same failure on other branches
|
pkg/ccl/serverccl/statusccl/statusccl_test.TestTenantStatusAPI failed with artifacts on release-22.2 @ 9efe3c5c73d3a5eb54b3c4bfe2e3a83504702c39:
Parameters: Same failure on other branches
|
pkg/ccl/serverccl/statusccl/statusccl_test.TestTenantStatusAPI failed with artifacts on release-22.2 @ 7cdf538ae6f0ff249d397ab64dbf144a543c5d1b:
Parameters: Same failure on other branches
|
pkg/ccl/serverccl/statusccl/statusccl_test.TestTenantStatusAPI failed with artifacts on release-22.2 @ beb24ea0688dcb96ca8a1f2ee85cd154efe0c0b2:
Parameters: Same failure on other branches
|
pkg/ccl/serverccl/statusccl/statusccl_test.TestTenantStatusAPI failed with artifacts on release-22.2 @ 6ab0496d81d303bae3854ee810fe2233ce18791f:
Parameters: Same failure on other branches
|
Informs cockroachdb#92382, cockroachdb#99770, cockroachdb#99559 Epic: none Release note: None
100083: kvserver: record metrics for ErrProposalDropped r=pavelkalinnikov a=tbg Touches #100096. Epic: none Release note: None 105093: sql: use datum alloc for crdb_internal stmt stats rows r=dt a=dt Happened to observe a cluster running a customer test suite which included a query that inspected stmt stats often, causing the CRDB node to spend a considerable amount of CPU time in production of the stmt stats vtable, in particular allocating (and then GC'ing) individual datums, especially given how wide this table has become with the addition of storage stats. This change uses a datum allocator to produce those rows to reduce the number of separate allocations from the runtime. Release note: none. Epic: none. 105197: statusccl: skip flaky TenantStatusAPI tests r=zachlite a=zachlite Informs #92382, #99770, #99559 Epic: none Release note: None Co-authored-by: Tobias Grieger <[email protected]> Co-authored-by: David Taylor <[email protected]> Co-authored-by: Zach Lite <[email protected]>
Informs cockroachdb#92382, cockroachdb#99770, cockroachdb#99559 Epic: none Release note: None
This test was mistakenly skipped in cockroachdb#105197, but the flake was addressed in cockroachdb#99054. The flake on the 22.2 release branch (tracked in cockroachdb#92382) was not novel, but un-treated. I misinterpreted this, and thought the skip on master would be for good measure, not realizing the flake was already treated. Epic: none Release note: None
106631: statusccl: unskip testTenantRanges pagination r=zachlite a=zachlite This test was mistakenly skipped in #105197, but the flake was addressed in #99054. The flake on the 22.2 release branch (tracked in #92382) was not novel, but untreated. I misinterpreted this, and thought the skip on master would be for good measure, not realizing the flake was already treated. Epic: none Release note: None Co-authored-by: zachlite <[email protected]>
This test was mistakenly skipped in #105197, but the flake was addressed in #99054. The flake on the 22.2 release branch (tracked in #92382) was not novel, but un-treated. I misinterpreted this, and thought the skip on master would be for good measure, not realizing the flake was already treated. Epic: none Release note: None
This is a 22.2 compatible work around for the flake fix introduced in cockroachdb#99054. In 22.2, crdb_internal.ranges is not yet compatible with secondary tenants, so instead we rely on TenantRanges itself with the `limit` param omitted. The method is the same: we count the number of ranges that belong to the tenant and only proceed when we know there are enough ranges for subsequent calls with pagination to succeed. Resolves cockroachdb#92382 Epic: none Release note: None
Resolved by #106710 |
pkg/ccl/serverccl/statusccl/statusccl_test.TestTenantStatusAPI failed with artifacts on release-22.2 @ f660a454f6781a379fcc2a17646dc859ac5af997:
Parameters:
TAGS=bazel,gss
Help
See also: How To Investigate a Go Test Failure (internal)
This test on roachdash | Improve this report!
Jira issue: CRDB-21732
The text was updated successfully, but these errors were encountered: