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

[bugfix] Prometheus metrics without warnings #9510

Merged

Conversation

pierresouchay
Copy link
Contributor

go-metrics is updated to 0.3.6 to properly handle help in prometheus metrics

This fixes #9303

go-metrics is updated to 0.3.6 to properly handle help in prometheus metrics

This fixes hashicorp#9303 and
hashicorp#9471
@pierresouchay pierresouchay force-pushed the prometheus_metrics_help_duplicate_fix branch from 8655951 to 408b249 Compare January 6, 2021 12:55
@pierresouchay
Copy link
Contributor Author

damned unstable tests:

  • TestACLResolver_Client

@pierresouchay
Copy link
Contributor Author

@mkcp Here is the fix with the patch hashicorp/go-metrics#122 included

Copy link
Contributor

@mkcp mkcp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent - appreciate you making a changelog note as well. Thanks again @pierresouchay !

@mkcp mkcp merged commit f0b7864 into hashicorp:master Jan 6, 2021
@hashicorp-ci
Copy link
Contributor

🍒 If backport labels were added before merging, cherry-picking will start automatically.

To retroactively trigger a backport after merging, add backport labels and re-run https://circleci.com/gh/hashicorp/consul/305409.

@hashicorp-ci
Copy link
Contributor

🍒✅ Cherry pick of commit f0b7864 onto release/1.9.x succeeded!

hashicorp-ci pushed a commit that referenced this pull request Jan 6, 2021
…uplicate_fix

[bugfix] Prometheus metrics without warnings
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants