Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix interface conversion in task state metrics #91

Closed
wants to merge 1 commit into from
Closed

Fix interface conversion in task state metrics #91

wants to merge 1 commit into from

Conversation

jacobjanco
Copy link

@jacobjanco jacobjanco commented Oct 25, 2018

Fixes:

panic: interface conversion: prometheus.Collector is *prometheus.GaugeVec, not *main.settableCounterVec

goroutine 12 [running]:
main.newMasterCollector.func20(0xc4202c4120, 0x8abfe0, 0xc42000e080, 0x0, 0x0)
	/go/src/github.com/mesos/mesos_exporter/master.go:419 +0x6f7
main.(*metricCollector).Collect(0xc42009eb30, 0xc4200ba420)
	/go/src/github.com/mesos/mesos_exporter/common.go:295 +0x1e7
github.com/mesos/mesos_exporter/vendor/github.com/prometheus/client_golang/prometheus.(*Registry).Gather.func2(0xc420024470, 0xc4200ba420, 0x8ac1e0, 0xc42009eb30)
	/go/src/github.com/mesos/mesos_exporter/vendor/github.com/prometheus/client_golang/prometheus/registry.go:383 +0x61
created by github.com/mesos/mesos_exporter/vendor/github.com/prometheus/client_golang/prometheus.(*Registry).Gather
	/go/src/github.com/mesos/mesos_exporter/vendor/github.com/prometheus/client_golang/prometheus/registry.go:381 +0x302

@jpeach
Copy link
Contributor

jpeach commented Oct 26, 2018

Sorry @jacobjanco, I merged #86 since it was posted earlier

@jpeach jpeach closed this Oct 26, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants