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

Configure metrics wrapper with the "global" object, not just the fanout. #9099

Merged
merged 5 commits into from
Jun 16, 2020

Conversation

mgritter
Copy link
Contributor

Configuration like service name wasn't being applied to new metrics.

@mgritter mgritter marked this pull request as draft May 28, 2020 16:23
@mgritter mgritter changed the title Change wrapper to use metrics.Metrics, not metrics.MetricSink. Configure metrics wrapper with the "global" object, not just the fanout. May 28, 2020
@mgritter mgritter marked this pull request as ready for review May 28, 2020 22:21
@mgritter mgritter added this to the 1.5 milestone May 28, 2020
Copy link
Contributor

@sgmiller sgmiller left a comment

Choose a reason for hiding this comment

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

LGTM!

@mgritter mgritter merged commit 6e2b91f into master Jun 16, 2020
catsby added a commit that referenced this pull request Jun 19, 2020
* master: (31 commits)
  changelog++
  changelog++
  Ui/replication status discoverability (#8705)
  Update CHANGELOG.md
  Counter that increments on every secret engine lease creation. (#9244)
  Add password_policy field to Azure docs (#9249)
  Replaced ClusterMetricSink's cluster name with an atomic.Value. (#9252)
  Fix database creds rotation panic for nil resp (#9258)
  changelog++
  changelog++
  Move sdk/helper/random -> helper/random (#9226)
  UI: Disallow kv2 with too large 'max versions' value (#9242)
  Allow mTLS for mysql secrets engine (#9181)
  docs: add sample revocation for mongodb (#9245)
  Add new Telemetry config options (#9238)
  Add a simple sealed gauge, updated when seal status changes (#9177)
  Test Shamir-to-Transit and Transit-to-Shamir Seal Migration for post-1.4 Vault. (#9214)
  Configure metrics wrapper with the "global" object, not just the fanout. (#9099)
  changelog++
  Add backend type to audit logs (#9167)
  ...
@mgritter mgritter deleted the new_metrics_4 branch June 24, 2020 00:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants