Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add summary metrics #1

Open
hawkw opened this issue Mar 4, 2023 · 0 comments
Open

add summary metrics #1

hawkw opened this issue Mar 4, 2023 · 0 comments
Labels
enhancement New feature or request

Comments

@hawkw
Copy link
Owner

hawkw commented Mar 4, 2023

~~it might be desirable to have some implementation of a Prometheus Gauge metric that's actually the current rolling average of measurements over some time window.

this would be intended for use cases where, for example, you read data from a sensor every second, but Prometheus only scrapes the device's metrics every ten seconds. in this case, the scrape is going to see just the data from the most recent read of the sensor, which may not accurately reflect the state of the world --- data from the other 9 reads of the sensor in that time window is completely discarded.

in this case, it might be desirable to expose a metric value that is actually an aggregate of multiple reads, so that the value seen by prometheus is less susceptible to random spikiness in the measurement?~~

EDIT: lol, this is basically how a prometheus Summary metric is supposed to be used. we should just add support for that...

@hawkw hawkw changed the title add a rolling average gauge add summary metrics Mar 4, 2023
@hawkw hawkw added the enhancement New feature or request label Mar 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant