
Merging this so main
reflects the deployed state.
@q3k, please review retrospectively anyway, if you have time and comments, so we can fix any crimes in a follow-up
So that's exactly what I though and why I started adding the counter implementation in the first place.
But it seems this isn't actually correct — prometheus only does the "account for…
My main motivation here was to not break the existing metric but keep consistency in the naming - a sem_pressure_volts
would not make sense but having sem_pressure_mbar
alongside `sem_pirani_vo…
As mentioned in the PR description, the counters only makes sense to keep if we change them to represent the total operating time with persistence on the succbone itself. Without that, the…