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.

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

sql/stmtdiagnostics: TestDiagnosticsRequest failed #103253

Closed
cockroach-teamcity opened this issue May 13, 2023 · 3 comments
Closed

sql/stmtdiagnostics: TestDiagnosticsRequest failed #103253

cockroach-teamcity opened this issue May 13, 2023 · 3 comments
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
Milestone

Comments

@cockroach-teamcity
Copy link
Member

cockroach-teamcity commented May 13, 2023

sql/stmtdiagnostics.TestDiagnosticsRequest failed with artifacts on release-23.1 @ b4533bdbc4b478f0ad311bad80b62bd072cf61cf:

=== RUN   TestDiagnosticsRequest
    test_log_scope.go:161: test logs captured to: /artifacts/tmp/_tmp/cb4e70d2cc4a901d2a7689654ffd0100/logTestDiagnosticsRequest1603224081
    test_log_scope.go:79: use -show-logs to present logs inline
=== CONT  TestDiagnosticsRequest
    statement_diagnostics_test.go:62: 
        	Error Trace:	github.com/cockroachdb/cockroach/pkg/sql/stmtdiagnostics_test/pkg/sql/stmtdiagnostics/statement_diagnostics_test.go:62
        	            				github.com/cockroachdb/cockroach/pkg/sql/stmtdiagnostics_test/pkg/sql/stmtdiagnostics/statement_diagnostics_test.go:82
        	            				github.com/cockroachdb/cockroach/pkg/sql/stmtdiagnostics_test/pkg/sql/stmtdiagnostics/statement_diagnostics_test.go:327
        	            				github.com/cockroachdb/cockroach/pkg/testutils/soon.go:71
        	            				github.com/cockroachdb/cockroach/pkg/util/retry/retry.go:213
        	            				github.com/cockroachdb/cockroach/pkg/testutils/soon.go:77
        	            				github.com/cockroachdb/cockroach/pkg/testutils/soon.go:59
        	            				github.com/cockroachdb/cockroach/pkg/testutils/soon.go:41
        	            				github.com/cockroachdb/cockroach/pkg/sql/stmtdiagnostics_test/pkg/sql/stmtdiagnostics/statement_diagnostics_test.go:296
        	Error:      	Should be false
        	Test:       	TestDiagnosticsRequest
        	Messages:   	request was completed and should have been removed from the registry
=== CONT  TestDiagnosticsRequest
    panic.go:522: -- test log scope end --
test logs left over in: /artifacts/tmp/_tmp/cb4e70d2cc4a901d2a7689654ffd0100/logTestDiagnosticsRequest1603224081
--- FAIL: TestDiagnosticsRequest (10.90s)
=== RUN   TestDiagnosticsRequest/ongoing_request_canceled/conditional
=== CONT  TestDiagnosticsRequest/ongoing_request_canceled/conditional
    testing.go:1343: test executed panic(nil) or runtime.Goexit: subtest may have called FailNow on a parent test
        --- FAIL: TestDiagnosticsRequest/ongoing_request_canceled/conditional (0.43s)
=== RUN   TestDiagnosticsRequest/ongoing_request_canceled
=== CONT  TestDiagnosticsRequest/ongoing_request_canceled
    testing.go:1343: test executed panic(nil) or runtime.Goexit: subtest may have called FailNow on a parent test
    --- FAIL: TestDiagnosticsRequest/ongoing_request_canceled (0.43s)

Parameters: TAGS=bazel,gss,deadlock

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-27904

@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 May 13, 2023
@cockroach-teamcity cockroach-teamcity added this to the 23.1 milestone May 13, 2023
@cockroach-teamcity cockroach-teamcity added this to Triage in SQL Queries May 13, 2023
@blathers-crl blathers-crl bot added the T-sql-queries SQL Queries Team label May 13, 2023
@cockroach-teamcity
Copy link
Member Author

sql/stmtdiagnostics.TestDiagnosticsRequest failed with artifacts on release-23.1 @ 6804aba73aecf63e0bfc7b648b9883d0286f7173:

=== RUN   TestDiagnosticsRequest
    test_log_scope.go:161: test logs captured to: /artifacts/tmp/_tmp/cb4e70d2cc4a901d2a7689654ffd0100/logTestDiagnosticsRequest4189300529
    test_log_scope.go:79: use -show-logs to present logs inline
=== CONT  TestDiagnosticsRequest
    statement_diagnostics_test.go:62: 
        	Error Trace:	github.com/cockroachdb/cockroach/pkg/sql/stmtdiagnostics_test/pkg/sql/stmtdiagnostics/statement_diagnostics_test.go:62
        	            				github.com/cockroachdb/cockroach/pkg/sql/stmtdiagnostics_test/pkg/sql/stmtdiagnostics/statement_diagnostics_test.go:82
        	            				github.com/cockroachdb/cockroach/pkg/sql/stmtdiagnostics_test/pkg/sql/stmtdiagnostics/statement_diagnostics_test.go:327
        	            				github.com/cockroachdb/cockroach/pkg/testutils/soon.go:71
        	            				github.com/cockroachdb/cockroach/pkg/util/retry/retry.go:213
        	            				github.com/cockroachdb/cockroach/pkg/testutils/soon.go:77
        	            				github.com/cockroachdb/cockroach/pkg/testutils/soon.go:59
        	            				github.com/cockroachdb/cockroach/pkg/testutils/soon.go:41
        	            				github.com/cockroachdb/cockroach/pkg/sql/stmtdiagnostics_test/pkg/sql/stmtdiagnostics/statement_diagnostics_test.go:296
        	Error:      	Should be false
        	Test:       	TestDiagnosticsRequest
        	Messages:   	request was completed and should have been removed from the registry
=== CONT  TestDiagnosticsRequest
    panic.go:522: -- test log scope end --
test logs left over in: /artifacts/tmp/_tmp/cb4e70d2cc4a901d2a7689654ffd0100/logTestDiagnosticsRequest4189300529
--- FAIL: TestDiagnosticsRequest (10.89s)
=== RUN   TestDiagnosticsRequest/ongoing_request_canceled
=== CONT  TestDiagnosticsRequest/ongoing_request_canceled
    testing.go:1343: test executed panic(nil) or runtime.Goexit: subtest may have called FailNow on a parent test
    --- FAIL: TestDiagnosticsRequest/ongoing_request_canceled (0.48s)
=== RUN   TestDiagnosticsRequest/ongoing_request_canceled/conditional
=== CONT  TestDiagnosticsRequest/ongoing_request_canceled/conditional
    testing.go:1343: test executed panic(nil) or runtime.Goexit: subtest may have called FailNow on a parent test
        --- FAIL: TestDiagnosticsRequest/ongoing_request_canceled/conditional (0.48s)

Parameters: TAGS=bazel,gss,deadlock

Help

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

This test on roachdash | Improve this report!

@msirek
Copy link
Contributor

msirek commented May 15, 2023

Unable to reproduce this with:

bazel test //pkg/sql/stmtdiagnostics:stmtdiagnostics_test --test_env=COCKROACH_NIGHTLY_STRESS=true --test_env=GOTRACEBACK=all --test_timeout=3660 --test_arg=-test.timeout=3655s --run_under '@com_github_cockroachdb_stress//:stress -bazel -maxruns 100 -maxtime 1h0m0s -maxfails 1 -p 4' --define gotags=bazel,gss,deadlock --nocache_test_results --test_output streamed --test_sharding_strategy=disabled --jobs 4  --build_event_binary_file=/tmp/beplog

@msirek
Copy link
Contributor

msirek commented May 21, 2023

Closing
Unable to reproduce this failure, and this is the first time this specific flavor of this test failure has occurred.
If it recurs maybe someone can think of a creative way to reproduce it.

@msirek msirek closed this as completed May 21, 2023
SQL Queries automation moved this from Triage to Done May 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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
Archived in project
Development

No branches or pull requests

2 participants