Pinned Metrics

[chipshort]

related to #1912

[webmaster128]

If errors lead to not updating the values and otherwise ignore them (which is fine), can we write the code in a way like:

1. If GetMetrics() succeeds, perform updates. Otherwise skip updates.
2. If GetPinnedMetrics() succeeds, perform updates. Otherwise skip updates.

This extends the old behaviour to more blocks

[chipshort]

you mean like this? (see latest commit)
Or do you mean: if one of them errors, update nothing

[webmaster128]

Exactly, I like. If no updates are coming to prometheus you can do the debugging. No need for an hard exit here.

[codecov]

Codecov Report

Attention: Patch coverage is 12.50000% with 14 lines in your changes missing coverage. Please review.

Project coverage is 54.67%. Comparing base (8171d65) to head (b3e8552).
Report is 4 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1922      +/-   ##
==========================================
- Coverage   54.69%   54.67%   -0.02%     
==========================================
  Files          65       65              
  Lines        9859     9865       +6     
==========================================
+ Hits         5392     5394       +2     
- Misses       3919     3923       +4     
  Partials      548      548              

Files	Coverage Δ
x/wasm/types/wasmer_engine.go	0.00% <ø> (ø)
x/wasm/keeper/metrics.go	47.05% <12.50%> (-2.95%)	⬇️