We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
By default, the prometheus exporter appends timestamps to metrics.
This goes against the current guidance from OpenMetrics about whether to expose timestamps.
We should consider not adding timestamps by default.
The text was updated successfully, but these errors were encountered:
This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 14 days.
Sorry, something went wrong.
@dyladan @legendecas Hi, I'd be happy to pick this up!
I think technically, our exporter uses the Prometheus text based exposition format as required by the spec in favor of OpenMetrics, but the same guidance is also present there.
@JacksonWeber thanks for picking this one up, it's yours 🙂
JacksonWeber
Successfully merging a pull request may close this issue.
By default, the prometheus exporter appends timestamps to metrics.
This goes against the current guidance from OpenMetrics about whether to expose timestamps.
We should consider not adding timestamps by default.
The text was updated successfully, but these errors were encountered: