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 objectives to summaries from struct tag #24

Merged
merged 2 commits into from
Oct 10, 2019

Conversation

mapno
Copy link
Contributor

@mapno mapno commented Oct 10, 2019

Prometheus has deprecated the default values for objectives and they now need to be explicitly specified.
Default values are those that client_golang library had before deprecation in v1.
If no objectives are specified, default objectives are used.

Solves https://github.com/cabify/gotoprom/issues/20.

@mapno mapno requested a review from colega October 10, 2019 09:15
@mapno mapno self-assigned this Oct 10, 2019
@mapno mapno force-pushed the pass-objectives-to-summaries branch 2 times, most recently from 38678ae to 0dba6ec Compare October 10, 2019 10:09
@mapno mapno force-pushed the pass-objectives-to-summaries branch from 0dba6ec to ff8df00 Compare October 10, 2019 10:44
Prometheus has deprecated the default values for objectives and they now need to be explicitly specified.
Default values are those that client_golang library had before deprecation in v1.
If no objectives are specified, default objectives are used.
@mapno mapno force-pushed the pass-objectives-to-summaries branch from ff8df00 to e5ec03d Compare October 10, 2019 10:48
@mapno mapno merged commit b3bd774 into master Oct 10, 2019
@colega colega deleted the pass-objectives-to-summaries branch January 10, 2020 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants