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: TestLogic_testserver_test_22_1_22_2 flaked due to empty conn string (master, x86 build) #94871

Closed
msbutler opened this issue Jan 7, 2023 · 1 comment · Fixed by #95211
Assignees
Labels
C-test-failure Broken test (automatically or manually discovered). skipped-test T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions)

Comments

@msbutler
Copy link
Collaborator

msbutler commented Jan 7, 2023

See flake here.

Stack trace:

No pass/skip/fail event found for test
=== RUN   TestLogic_testserver_test_22_1_22_2
    test_log_scope.go:161: test logs captured to: /artifacts/tmp/_tmp/a0cd2ed7268f1ad03abb8f1d57f5fcc9/logTestLogic_testserver_test_22_1_22_22661895086
    test_log_scope.go:79: use -show-logs to present logs inline
panic: empty connection string

goroutine 94 [running]:
github.com/cockroachdb/cockroach-go/v2/testserver.(*testServerImpl).pollListeningURLFile(0xc0002fc900, 0x0)
	github.com/cockroachdb/cockroach-go/v2/testserver/external/com_github_cockroachdb_cockroach_go_v2/testserver/testserver.go:722 +0x765
github.com/cockroachdb/cockroach-go/v2/testserver.(*testServerImpl).StartNode.func1()
	github.com/cockroachdb/cockroach-go/v2/testserver/external/com_github_cockroachdb_cockroach_go_v2/testserver/testservernode.go:108 +0x2f
created by github.com/cockroachdb/cockroach-go/v2/testserver.(*testServerImpl).StartNode
	github.com/cockroachdb/cockroach-go/v2/testserver/external/com_github_cockroachdb_cockroach_go_v2/testserver/testservernode.go:107 +0xcbb

Jira issue: CRDB-23194

@msbutler msbutler added C-test-failure Broken test (automatically or manually discovered). T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions) labels Jan 7, 2023
@msbutler
Copy link
Collaborator Author

msbutler commented Jan 9, 2023

another one .

Going to open a PR to skip.

msbutler added a commit to msbutler/cockroach that referenced this issue Jan 9, 2023
TestStoreCapacityAfterSplit, TestMergeQueueDoesNotInterruptReplicationChange,
testserver_test_22.1_22.2, testserve_upgrade_node.

Informs: cockroachdb#94871, cockroachdb#94951, cockroachdb#92677 cockroachdb#94956

Release note: None
craig bot pushed a commit that referenced this issue Jan 9, 2023
94954: kv/server: skip a bunch of flakey tests r=adityamaru a=msbutler

TestStoreCapacityAfterSplit, TestMergeQueueDoesNotInterruptReplicationChange,
testserver_test_22.1_22.2, testserver_upgrade_node.

Informs: #94871, #94951, #92677, #94956

Release note: None

Co-authored-by: Michael Butler <[email protected]>
@craig craig bot closed this as completed in f273aae Jan 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-test-failure Broken test (automatically or manually discovered). skipped-test T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants