Skip to content

Remove Consul test goroutine as it is causing panics

Reuben Pereira requested to merge rp/fix-consul-panic into master

What does this MR do and why?

Describe in detail what your merge request does and why.

Content

  • Remove Consul test goroutine as it is causing panics, which causes the pods to terminate and restart every 5 minutes.
[signal SIGSEGV: segmentation violation code=0x1 addr=0x30 pc=0x7bf52a]

goroutine 61 [running]:
gitlab.com/gitlab-org/release-tools/metrics/internal/metrics/consul.(*Client).GetKeyValue(0xc00012cb60, {0x8e74c4, 0x8})
	/source/internal/metrics/consul/consul.go:81 +0x1ca
main.TryConsul(0xc00022a340, 0xc00021a658, 0xc000226480)
	/source/main.go:69 +0x585
created by main.main in goroutine 1
	/source/main.go:135 +0x825

Log entry

The panic seems to be coming from here: internal/metrics/consul/consul.go#L81

Author Check-list

  • Has documentation been updated?
Edited by Reuben Pereira

Merge request reports