fix: metrics were not registered correctly, this simplifies the setup

entur / lamassu

Mobility hub

European Union Public License 1.2

5 stars 7 forks source link

fix: metrics were not registered correctly, this simplifies the setup #439

Closed testower closed 2 months ago

testower commented 2 months ago

Summary

Fixes issues with registering metrics. Metrics are registered once per combination of name and tags - subsequent calls are lost. It is required to keep counters for each combination, which gets hairy. This PR simplifies the setup, removes the file error metrics, and removes version and file tags.

Issue

Closes #438

Unit tests

Unit tests were added to reproduce the problems and verify solution.

sonarcloud[bot] commented 2 months ago

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud