pkg/ccl/logictestccl/tests/3node-tenant-multiregion/3node-tenant-multiregion_test: TestTenantExecBuild_distsql_tenant_locality failed #118627
Assignees
Labels
branch-master
Failures on the master branch.
C-test-failure
Broken test (automatically or manually discovered).
O-robot
Originated from a bot.
T-sql-queries
SQL Queries Team
Milestone
Comments
cockroach-teamcity
added
branch-master
|
sigh ok, I'll add the skip. |
michae2
removed
the
release-blocker
|
pkg/ccl/logictestccl/tests/3node-tenant-multiregion/3node-tenant-multiregion_test.TestTenantExecBuild_distsql_tenant_locality failed with artifacts on master @ 15961a19faca0e2b66df2d01a547549523ca70c7: Parameters:
|
|
pkg/ccl/logictestccl/tests/3node-tenant-multiregion/3node-tenant-multiregion_test.TestTenantExecBuild_distsql_tenant_locality failed with artifacts on master @ 3c41c509a87cba7a1fd3f5cfdb0f6badb78e3704: Parameters:
|
|
pkg/ccl/logictestccl/tests/3node-tenant-multiregion/3node-tenant-multiregion_test.TestTenantExecBuild_distsql_tenant_locality failed with artifacts on master @ d272e9ef5589deff570efc023db6c70edfde311c: Parameters:
|
rickystewart
added a commit
to rickystewart/cockroach
that referenced
this issue
Feb 5, 2024
Flaky test: see cockroachdb#118627 Epic: None Release note: None
rickystewart
added a commit
to rickystewart/cockroach
that referenced
this issue
Feb 5, 2024
Flaky test: see cockroachdb#118627 Epic: None Release note: None
rickystewart
added a commit
to rickystewart/cockroach
that referenced
this issue
Feb 5, 2024
Flaky test: see cockroachdb#118627 Epic: None Release note: None
craig bot
pushed a commit
that referenced
this issue
Feb 5, 2024
118575: schemachanger: Prep work for ALTER DATABASE ... CONFIGURE ZONE r=Xiang-Gu a=Xiang-Gu This PR contains three preparation commits for supporting zone configs in DSC. They should be rather uncontroversial and easy to review. See commit message for details. Informs: #117574 Epic: CRDB-31473 118679: sql: minor improvements around estimated row count r=yuzefovich a=yuzefovich This commit makes it so that we use the ceiling of the estimated row count in scans and aggregation. It also fixes recently-introduced bug that could cause integer overflow in the agg alloc code. Fixes: #118724. Epic: None Release note: None 118680: ui: change start and end values on stmt details charts r=maryliag a=maryliag Previously, we were using the data values to define the start and end date of charts, which could cause confusion, since users might think that is a bug of now showing data, when instead there is no data to show. This commit forces the start and end of the chart to be the selected period, this way we always show the correct period even if we don't have data to show. Fixes #102214 https://www.loom.com/share/3f463d82407b4ab186a5b0c1e1ab2369 Release note (ui change): On Statement Details page always show the entire selected period, instead of just the period that had data. 118769: logictest: skip `distsql_tenant_locality` under duress r=yuzefovich,michae2 a=rickystewart Flaky test: see #118627 Epic: None Release note: None Co-authored-by: Xiang Gu <xiang@cockroachlabs.com> Co-authored-by: Yahor Yuzefovich <yahor@cockroachlabs.com> Co-authored-by: maryliag <marylia@cockroachlabs.com> Co-authored-by: Ricky Stewart <ricky@cockroachlabs.com>
michae2
added a commit
to michae2/cockroach
that referenced
this issue
Feb 6, 2024
We already had a `skip` command in our fork of Test-script, but it only called `skip.IgnoreLint` (and wasn't used). Flesh it out a bit with options for `skip.UnderStress` and family. The new syntax comes in three forms: ``` skip <ISSUE> [args...] skip ignorelint [args...] skip under <deadlock/race/stress/stressrace/metamorphic/duress> [ISSUE] [args...] ``` The first form calls `skip.WithIssue`, the second form calls `skip.IgnoreLint`, and the third form calls `skip.UnderStress` and family. Also fix one mistaken use of `skip` which should have been `skipif`. Informs: cockroachdb#118627 Epic: None Release note: None
michae2
added a commit
to michae2/cockroach
that referenced
this issue
Feb 7, 2024
We already had a `skip` command in our fork of Test-script. Flesh it out a bit with options for `skip.UnderStress` and family. The new syntax comes in three forms: ``` skip <ISSUE> [args...] skip ignorelint [args...] skip under <deadlock/race/stress/stressrace/metamorphic/duress> [ISSUE] [args...] ``` The first form calls `skip.WithIssue`, the second form calls `skip.IgnoreLint`, and the third form calls `skip.UnderStress` and family. Before this change, the only supported form was `skip [ISSUE]` which called `skip.IgnoreLint`. With this cange, commands in this original form will instead call `skip.WithIssue`. AFAIK there are no existing skip commands currently, so this should be a moot point. Also fix one mistaken use of `skip` which should have been `skipif`. Informs: cockroachdb#118627 Epic: None Release note: None
michae2
added a commit
to michae2/cockroach
that referenced
this issue
Feb 7, 2024
In cockroachdb#118769 we added a skip under duress for the distsql_tenant_locality logic test due to cockroachdb#118627. Now that the `skip` command supports calling `skip.UnderDuress()`, move the skip for distsql_tenant_locality from the logictest template to the test script itself. Informs: cockroachdb#118627 Epic: None Release note: None
craig bot
pushed a commit
that referenced
this issue
Feb 7, 2024
118820: logictest: add ability to call skip.UnderStress from Test-script r=rafiss,rickystewart a=michae2 **logictest: add ability to call skip.UnderStress from Test-script** We already had a `skip` command in our fork of Test-script. Flesh it out a bit with options for `skip.UnderStress` and family. The new syntax comes in three forms: ``` skip <ISSUE> [args...] skip ignorelint [args...] skip under <deadlock/race/stress/stressrace/metamorphic/duress> [ISSUE] [args...] ``` The first form calls `skip.WithIssue`, the second form calls `skip.IgnoreLint`, and the third form calls `skip.UnderStress` and family. Before this change, the only supported form was `skip [ISSUE]` which called `skip.IgnoreLint`. With this cange, commands in this original form will instead call `skip.WithIssue`. AFAIK there are no existing skip commands currently, so this should be a moot point. Also fix one mistaken use of `skip` which should have been `skipif`. Informs: #118627 Epic: None Release note: None --- **exec: skip distsql_tenant_locality under duress** In #118769 we added a skip under duress for the distsql_tenant_locality logic test due to #118627. Now that the `skip` command supports calling `skip.UnderDuress()`, move the skip for distsql_tenant_locality from the logictest template to the test script itself. Informs: #118627 Epic: None Release note: None Co-authored-by: Michael Erickson <michae2@cockroachlabs.com>
wenyihu6
pushed a commit
to wenyihu6/cockroach
that referenced
this issue
Feb 21, 2024
Flaky test: see cockroachdb#118627 Epic: None Release note: None
wenyihu6
pushed a commit
to wenyihu6/cockroach
that referenced
this issue
Feb 21, 2024
We already had a `skip` command in our fork of Test-script. Flesh it out a bit with options for `skip.UnderStress` and family. The new syntax comes in three forms: ``` skip <ISSUE> [args...] skip ignorelint [args...] skip under <deadlock/race/stress/stressrace/metamorphic/duress> [ISSUE] [args...] ``` The first form calls `skip.WithIssue`, the second form calls `skip.IgnoreLint`, and the third form calls `skip.UnderStress` and family. Before this change, the only supported form was `skip [ISSUE]` which called `skip.IgnoreLint`. With this cange, commands in this original form will instead call `skip.WithIssue`. AFAIK there are no existing skip commands currently, so this should be a moot point. Also fix one mistaken use of `skip` which should have been `skipif`. Informs: cockroachdb#118627 Epic: None Release note: None
wenyihu6
pushed a commit
to wenyihu6/cockroach
that referenced
this issue
Feb 21, 2024
In cockroachdb#118769 we added a skip under duress for the distsql_tenant_locality logic test due to cockroachdb#118627. Now that the `skip` command supports calling `skip.UnderDuress()`, move the skip for distsql_tenant_locality from the logictest template to the test script itself. Informs: cockroachdb#118627 Epic: None Release note: None
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
pkg/ccl/logictestccl/tests/3node-tenant-multiregion/3node-tenant-multiregion_test.TestTenantExecBuild_distsql_tenant_locality failed with artifacts on master @ 0778173b902ce6d418b843389d20509b934f451d:
Parameters:
TAGS=bazel,gssstress=trueHelp
See also: How To Investigate a Go Test Failure (internal)
This test on roachdash | Improve this report!
Jira issue: CRDB-35814
The text was updated successfully, but these errors were encountered: