You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In the last PR that got merged regarding updating the workersRegistered metric (see #3076), I missed providing the instantiation of the GaugeVec to the emitter struct.
Naturally, this leads Prometheus scraping to make the whole web process die with a SIGSEGV:
The fix is rather simple - give the instantiated object to the PrometheusEmitter instance and that's fixed.
This issue is meant more to document how little things like this, at the moment, are able to greatly affect the execution of web instances - call out to #2896
The text was updated successfully, but these errors were encountered:
cirocosta
changed the title
metrics: nil pointer dereference for workersState in Prometheus emitter
nil pointer dereference for workersRegistered in Prometheus emitter
Mar 15, 2019
Hey,
In the last PR that got merged regarding updating the
workersRegistered
metric (see #3076), I missed providing the instantiation of theGaugeVec
to theemitter
struct.Naturally, this leads Prometheus scraping to make the whole
web
process die with a SIGSEGV:The fix is rather simple - give the instantiated object to the
PrometheusEmitter
instance and that's fixed.This issue is meant more to document how little things like this, at the moment, are able to greatly affect the execution of web instances - call out to #2896
The text was updated successfully, but these errors were encountered: