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

acceptance: TestDockerCLI/test_demo_node_cmds.tcl flaked [index out of range [6] with length 6] #96978

Closed
rafiss opened this issue Feb 10, 2023 · 1 comment
Labels
C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.

Comments

@rafiss
Copy link
Collaborator

rafiss commented Feb 10, 2023

Observed here: https://teamcity.cockroachdb.com/buildConfiguration/Cockroach_BazelEssentialCi/8664061?showRootCauses=false&expandBuildChangesSection=true&expandBuildProblemsSection=true&expandBuildTestsSection=true

Maybe relates to #96450 and #96239 and #96162

Relevant logs:

expect: does "\r\nroot@127.0.0.1:26257/defaultdb> " (spawn_id exp3) match glob pattern "defaultdb>"? yes
expect: set expect_out(0,string) "defaultdb>"
expect: set expect_out(spawn_id) "exp3"
expect: set expect_out(buffer) "\r\nroot@127.0.0.1:26257/defaultdb>"
send: sending "\demo add region=ca-central,zone=a\r" to { exp3 }

expect: does " " (spawn_id exp3) match glob pattern "node 6 has been added with locality "region=ca-central,zone=a""? no
root@127.0.0.1:26257/defaultdb> \demo add region=ca-central,zone=a

expect: does " \demo add region=ca-central,zone=a\r\n" (spawn_id exp3) match glob pattern "node 6 has been added with locality "region=ca-central,zone=a""? no
*
* ERROR: a panic has occurred!
* runtime error: index out of range [6] with length 6
* (1) attached stack trace
*   -- stack trace:
*   | runtime.gopanic
*   | 	GOROOT/src/runtime/panic.go:890
*   | runtime.goPanicIndex
*   | 	GOROOT/src/runtime/panic.go:113
*   | github.com/cockroachdb/cockroach/pkg/cli/democluster.(*transientCluster).GetLocality
*   | 	github.com/cockroachdb/cockroach/pkg/cli/democluster/demo_cluster.go:1858
*   | github.com/cockroachdb/cockroach/pkg/cli/clisqlshell.(*cliState).handleDemoAddNode
*   | 	github.com/cockroachdb/cockroach/pkg/cli/clisqlshell/sql.go:682
*   | github.com/cockroachdb/cockroach/pkg/cli/clisqlshell.(*cliState).handleDemo
*   | 	github.com/cockroachdb/cockroach/pkg/cli/clisqlshell/sql.go:664
*   | github.com/cockroachdb/cockroach/pkg/cli/clisqlshell.(*cliState).doHandleCliCmd
*   | 	github.com/cockroachdb/cockroach/pkg/cli/clisqlshell/sql.go:1442
*   | github.com/cockroachdb/cockroach/pkg/cli/clisqlshell.(*cliState).doRunShell
*   | 	github.com/cockroachdb/cockroach/pkg/cli/clisqlshell/sql.go:2157
*   | github.com/cockroachdb/cockroach/pkg/cli/clisqlshell.(*cliState).RunInteractive
*   | 	github.com/cockroachdb/cockroach/pkg/cli/clisqlshell/sql.go:2106
*   | github.com/cockroachdb/cockroach/pkg/cli/clisqlcfg.(*Context).Run
*   | 	github.com/cockroachdb/cockroach/pkg/cli/clisqlcfg/context.go:225
*   | github.com/cockroachdb/cockroach/pkg/cli.runDemoInternal
*   | 	github.com/cockroachdb/cockroach/pkg/cli/demo.go:381
*   | github.com/cockroachdb/cockroach/pkg/cli.runDemo

Jira issue: CRDB-24444

@rafiss rafiss added the C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. label Feb 10, 2023
@rafiss rafiss added this to To do in SQL CLI (demo + sql + workload + dump) via automation Feb 10, 2023
@rafiss
Copy link
Collaborator Author

rafiss commented Jul 7, 2023

closing in favor of #102257

@rafiss rafiss closed this as completed Jul 7, 2023
SQL CLI (demo + sql + workload + dump) automation moved this from To do to Done Jul 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.
Development

No branches or pull requests

1 participant