Thanks a lot for contribution, I wonder how it works before without errors

Thanks a lot for contribution, I wonder how it works before without errors

it does error (albeit silently), there is only a check if err == nill and so if there is one the err is never pushed to the upstream func to report it.

it does error (albeit silently), there is only a check if err == nill and so if there is one the err is never pushed to the upstream func to report it.

but, this numbers is present in our Grafana Dashboard and show 24x7, correcly now

it does error (albeit silently), there is only a check if err == nill and so if there is one the err is never pushed to the upstream func to report it.

but, this numbers is present in our Grafana Dashboard and show 24x7, correcly now

after some investigation, it. look like the new code never worked, this metrics as been broken since 0.17.0, last working docker image for the metrics-exporter that work is the 0.16.1.

28af2b0 is the commit when it started to never get a value other than 0

Looking at the history it look like per default the image for the metrics-exporter is shipped with :latest but this tag was not in sync with the master branch before ?

@adu-bricks thanks a lot for detailed investigation!

@adu-bricks could you check and confirm issue fixed on metrics-exporter 0.18.0 and latest image version?

@adu-bricks could you check and confirm issue fixed on metrics-exporter 0.18.0 and latest image version?

I can confirm pulling the news :latest and :0.18.0 image fixed the metrics issues.

Thanks !