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

sql: multiregion-9node-3region-3azs-vec-off query has no home region #102406

Closed
cockroach-teamcity opened this issue Apr 27, 2023 · 1 comment · Fixed by #102484
Closed

sql: multiregion-9node-3region-3azs-vec-off query has no home region #102406

cockroach-teamcity opened this issue Apr 27, 2023 · 1 comment · Fixed by #102484
Assignees
@msirek
Labels
branch-release-23.1 Used to mark GA and release blockers, technical advisories, and bugs for 23.1 C-test-failure Broken test (automatically or manually discovered). O-robot Originated from a bot. T-sql-queries SQL Queries Team
Milestone
23.1

Comments

@cockroach-teamcity
Copy link
Member

cockroach-teamcity commented Apr 27, 2023 •
edited by cockroach-jira-scripts
Loading

pkg/ccl/logictestccl/tests/multiregion-9node-3region-3azs-vec-off/multiregion-9node-3region-3azs-vec-off_test.TestCCLLogic_multi_region_remote_access_error failed with artifacts on release-23.1 @ 6636edf9d8c0ebf9cdeedeeffd563ecb49da5b8a:

=== RUN   TestCCLLogic_multi_region_remote_access_error
    test_log_scope.go:161: test logs captured to: /artifacts/tmp/_tmp/875622737c3d90631dff1d5650587860/logTestCCLLogic_multi_region_remote_access_error1552426472
    test_log_scope.go:79: use -show-logs to present logs inline
*
* INFO: Running test with the default test tenant. If you are only seeing a test case failure when this message appears, there may be a problem with your test case running within tenants.
*
*
* INFO: Running test with the default test tenant. If you are only seeing a test case failure when this message appears, there may be a problem with your test case running within tenants.
*
*
* INFO: Running test with the default test tenant. If you are only seeing a test case failure when this message appears, there may be a problem with your test case running within tenants.
*
*
* INFO: Running test with the default test tenant. If you are only seeing a test case failure when this message appears, there may be a problem with your test case running within tenants.
*
*
* INFO: Running test with the default test tenant. If you are only seeing a test case failure when this message appears, there may be a problem with your test case running within tenants.
*
*
* INFO: Running test with the default test tenant. If you are only seeing a test case failure when this message appears, there may be a problem with your test case running within tenants.
*
*
* INFO: Running test with the default test tenant. If you are only seeing a test case failure when this message appears, there may be a problem with your test case running within tenants.
*
*
* INFO: Running test with the default test tenant. If you are only seeing a test case failure when this message appears, there may be a problem with your test case running within tenants.
*
*
* INFO: Running test with the default test tenant. If you are only seeing a test case failure when this message appears, there may be a problem with your test case running within tenants.
*
[08:09:58] --- progress: /home/roach/.cache/bazel/_bazel_roach/c5a4e7d36696d9cd970af2045211a7df/sandbox/processwrapper-sandbox/2245/execroot/com_github_cockroachdb_cockroach/bazel-out/k8-fastbuild/bin/pkg/ccl/logictestccl/tests/multiregion-9node-3region-3azs-vec-off/multiregion-9node-3region-3azs-vec-off_test_/multiregion-9node-3region-3azs-vec-off_test.runfiles/com_github_cockroachdb_cockroach/pkg/ccl/logictestccl/testdata/logic_test/multi_region_remote_access_error: 17 statements
[08:10:00] --- progress: /home/roach/.cache/bazel/_bazel_roach/c5a4e7d36696d9cd970af2045211a7df/sandbox/processwrapper-sandbox/2245/execroot/com_github_cockroachdb_cockroach/bazel-out/k8-fastbuild/bin/pkg/ccl/logictestccl/tests/multiregion-9node-3region-3azs-vec-off/multiregion-9node-3region-3azs-vec-off_test_/multiregion-9node-3region-3azs-vec-off_test.runfiles/com_github_cockroachdb_cockroach/pkg/ccl/logictestccl/testdata/logic_test/multi_region_remote_access_error: 35 statements
[08:10:05] --- progress: /home/roach/.cache/bazel/_bazel_roach/c5a4e7d36696d9cd970af2045211a7df/sandbox/processwrapper-sandbox/2245/execroot/com_github_cockroachdb_cockroach/bazel-out/k8-fastbuild/bin/pkg/ccl/logictestccl/tests/multiregion-9node-3region-3azs-vec-off/multiregion-9node-3region-3azs-vec-off_test_/multiregion-9node-3region-3azs-vec-off_test.runfiles/com_github_cockroachdb_cockroach/pkg/ccl/logictestccl/testdata/logic_test/multi_region_remote_access_error: 40 statements
    logic.go:2871: 
         
        /home/roach/.cache/bazel/_bazel_roach/c5a4e7d36696d9cd970af2045211a7df/sandbox/processwrapper-sandbox/2245/execroot/com_github_cockroachdb_cockroach/bazel-out/k8-fastbuild/bin/pkg/ccl/logictestccl/tests/multiregion-9node-3region-3azs-vec-off/multiregion-9node-3region-3azs-vec-off_test_/multiregion-9node-3region-3azs-vec-off_test.runfiles/com_github_cockroachdb_cockroach/pkg/ccl/logictestccl/testdata/logic_test/multi_region_remote_access_error:343: WITH vtab AS (SELECT * FROM messages_rbt)
        SELECT vtab.account_id, vtab.message FROM vtab
          INNER LOOKUP JOIN messages_rbr rbr on vtab.account_id = rbr.account_id and rbr.crdb_region = 'ap-southeast-2'
        expected success, but found
        (XCHR2) Query has no home region. Try adding a LIMIT clause. For more information, see https://www.cockroachlabs.com/docs/stable/cost-based-optimizer.html#control-whether-queries-are-limited-to-a-single-region
        relational.go:2118: in buildDistribute()
[08:10:05] --- done: /home/roach/.cache/bazel/_bazel_roach/c5a4e7d36696d9cd970af2045211a7df/sandbox/processwrapper-sandbox/2245/execroot/com_github_cockroachdb_cockroach/bazel-out/k8-fastbuild/bin/pkg/ccl/logictestccl/tests/multiregion-9node-3region-3azs-vec-off/multiregion-9node-3region-3azs-vec-off_test_/multiregion-9node-3region-3azs-vec-off_test.runfiles/com_github_cockroachdb_cockroach/pkg/ccl/logictestccl/testdata/logic_test/multi_region_remote_access_error with config multiregion-9node-3region-3azs-vec-off: 43 tests, 1 failures
    logic.go:3983: 
        /home/roach/.cache/bazel/_bazel_roach/c5a4e7d36696d9cd970af2045211a7df/sandbox/processwrapper-sandbox/2245/execroot/com_github_cockroachdb_cockroach/bazel-out/k8-fastbuild/bin/pkg/ccl/logictestccl/tests/multiregion-9node-3region-3azs-vec-off/multiregion-9node-3region-3azs-vec-off_test_/multiregion-9node-3region-3azs-vec-off_test.runfiles/com_github_cockroachdb_cockroach/pkg/ccl/logictestccl/testdata/logic_test/multi_region_remote_access_error:349: error while processing
    logic.go:3983: /home/roach/.cache/bazel/_bazel_roach/c5a4e7d36696d9cd970af2045211a7df/sandbox/processwrapper-sandbox/2245/execroot/com_github_cockroachdb_cockroach/bazel-out/k8-fastbuild/bin/pkg/ccl/logictestccl/tests/multiregion-9node-3region-3azs-vec-off/multiregion-9node-3region-3azs-vec-off_test_/multiregion-9node-3region-3azs-vec-off_test.runfiles/com_github_cockroachdb_cockroach/pkg/ccl/logictestccl/testdata/logic_test/multi_region_remote_access_error:349: too many errors encountered, skipping the rest of the input
    panic.go:522: -- test log scope end --
test logs left over in: /artifacts/tmp/_tmp/875622737c3d90631dff1d5650587860/logTestCCLLogic_multi_region_remote_access_error1552426472
--- FAIL: TestCCLLogic_multi_region_remote_access_error (20.55s)

Parameters: TAGS=bazel,gss

Help

See also: How To Investigate a Go Test Failure (internal)

/cc @cockroachdb/sql-queries

This test on roachdash | Improve this report!

Jira issue: CRDB-27427
@cockroach-teamcity cockroach-teamcity added branch-release-23.1 Used to mark GA and release blockers, technical advisories, and bugs for 23.1 C-test-failure Broken test (automatically or manually discovered). O-robot Originated from a bot. labels Apr 27, 2023
@cockroach-teamcity cockroach-teamcity added this to the 23.1 milestone Apr 27, 2023
@blathers-crl blathers-crl bot added the T-sql-queries SQL Queries Team label Apr 27, 2023
@rharding6373
Copy link
Collaborator

This test was added by #100977 , CC @msirek

@rharding6373 rharding6373 changed the title pkg/ccl/logictestccl/tests/multiregion-9node-3region-3azs-vec-off/multiregion-9node-3region-3azs-vec-off_test: TestCCLLogic_multi_region_remote_access_error failed sql: multiregion-9node-3region-3azs-vec-off query has no home region Apr 27, 2023
@msirek msirek self-assigned this Apr 27, 2023
@msirek msirek mentioned this issue Apr 27, 2023
Merged
craig bot pushed a commit that referenced this issue Apr 27, 2023
@msbutler
Merge #102189 #102484
79f42d2 
102189: c2c: clean up stream ingestion completion on resume r=lidorcarmel a=msbutler

This patch ensures that the stream ingestion job completes properly on
resumption after the user established a cutover timestamp while the job was
paused.

Fixes #102158

Release note: none

102484: logictestccl: add missing retries to multi_region_remote_access_error test r=rharding6373 a=msirek

PR #102103 added 2 tests to multi_region_remote_access_error which require retries in case zone configs did not propagate correctly.

This adds the retries.

Fixes #102406
Fixes #102292

Release note: None

Co-authored-by: Michael Butler <butler@cockroachlabs.com>
Co-authored-by: Mark Sirek <sirek@cockroachlabs.com>
@craig craig bot closed this as completed in d482c2d Apr 27, 2023
@blathers-crl blathers-crl bot mentioned this issue Apr 27, 2023
Merged
blathers-crl bot pushed a commit that referenced this issue Apr 27, 2023
logictestccl: add missing retries to multi_region_remote_access_error…
fc6f7c9 
… test

PR #102103 added 2 tests to multi_region_remote_access_error which require
retries in case zone configs did not propagate correctly.

This adds the retries.

Fixes #102406

Release note: None
cameronnunez pushed a commit to cameronnunez/cockroach that referenced this issue May 2, 2023
@cameronnunez
logictestccl: add missing retries to multi_region_remote_access_error…
a737ee0 
… test

PR cockroachdb#102103 added 2 tests to multi_region_remote_access_error which require
retries in case zone configs did not propagate correctly.

This adds the retries.

Fixes cockroachdb#102406

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

@msirek msirek

Labels
branch-release-23.1 Used to mark GA and release blockers, technical advisories, and bugs for 23.1 C-test-failure Broken test (automatically or manually discovered). O-robot Originated from a bot. T-sql-queries SQL Queries Team
Projects
SQL Queries
Archived in project
Milestone
23.1
Development

Successfully merging a pull request may close this issue.

logictestccl: add missing retries to multi_region_remote_access_error test msirek/cockroach
3 participants
@msirek @rharding6373 @cockroach-teamcity