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

Bug Report: vttablet panic on shutdown #14458

Closed
Tracked by #13990
deepthi opened this issue Nov 3, 2023 · 0 comments · Fixed by #14457
Closed
Tracked by #13990

Bug Report: vttablet panic on shutdown #14458

deepthi opened this issue Nov 3, 2023 · 0 comments · Fixed by #14457
Labels
Component: General Changes throughout the code base Type: Bug
Milestone

Comments

@deepthi
Copy link
Member

deepthi commented Nov 3, 2023

Overview of the Issue

vttablet panics on shutdown.

Reproduction Steps

Edit examples/local/401_teardown.sh and comment out line 69 that deletes $VTDATAROOT.

$ ./101_initial_cluster.sh
...
$ ./401_teardown.sh

Observe the panic in any of the vttablet log files under $VTDATAROOT/tmp

Binary Version

v18

Operating System and Environment details

Any

Log Fragments

panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x2 addr=0x30 pc=0x1057df668]

goroutine 983 [running]:
vitess.io/vitess/go/vt/topo.(*Server).GetCellInfo(0x14000b14ac0, {0x1073ddde0, 0x140006f7500}, {0x16b0e3216, 0x9}, 0x0)
	vitess.io/vitess/go/vt/topo/cell_info.go:71 +0xd8
vitess.io/vitess/go/vt/topo.(*Server).ConnForCell(0x14000b14ac0, {0x1073ddde0?, 0x140006f7500?}, {0x16b0e3216, 0x9})
	vitess.io/vitess/go/vt/topo/server.go:269 +0x108
vitess.io/vitess/go/vt/topo.(*Server).GetTablet(0x140009afd40?, {0x1073ddde0, 0x140006f7500}, 0x14000b14000)
	vitess.io/vitess/go/vt/topo/tablet.go:228 +0x4c
vitess.io/vitess/go/vt/topo.(*Server).UpdateTabletFields(0x1073ddc78?, {0x1073ddde0?, 0x140006f7500?}, 0x14000b14000, 0x14000ae4f40)
	vitess.io/vitess/go/vt/topo/tablet.go:393 +0x168
vitess.io/vitess/go/vt/vttablet/tabletmanager.(*TabletManager).Close(0x140004d01a0)
	vitess.io/vitess/go/vt/vttablet/tabletmanager/tm_init.go:437 +0x98
vitess.io/vitess/go/cmd/vttablet/cli.run.func1()
	vitess.io/vitess/go/cmd/vttablet/cli/cli.go:162 +0x2c
vitess.io/vitess/go/event.(*Hooks).Fire.func1(0x140010e4000?)
	vitess.io/vitess/go/event/hooks.go:49 +0x2c
created by vitess.io/vitess/go/event.(*Hooks).Fire in goroutine 1
	vitess.io/vitess/go/event/hooks.go:48 +0xe8
@deepthi deepthi added Type: Bug Needs Triage This issue needs to be correctly labelled and triaged Component: General Changes throughout the code base and removed Needs Triage This issue needs to be correctly labelled and triaged labels Nov 3, 2023
@deepthi deepthi added this to the v18.0.0 milestone Nov 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: General Changes throughout the code base Type: Bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant