-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Fix builder.Builder WithMetrics signature #1455
Fix builder.Builder WithMetrics signature #1455
Conversation
The `WithMetrics` method signature changed from ``` func (b *Builder) WithMetrics(r *prometheus.Registry) { ``` to ``` func (b *Builder) WithMetrics(r prometheus.Registerer) { ``` in kubernetes#1223 but one occurrence was forgotten. Signed-off-by: Lénaïc Huard <lenaic.huard@datadoghq.com>
Welcome @L3n41c! |
/assign @tariq1890 |
/lgtm Thanks! @L3n41c |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: L3n41c, mrueg, tariq1890 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Thank you! We should backport this to release-2.0 as I suspect it has the same problem? |
Backport #1455: Fix builder.Builder WithMetrics signature
What this PR does / why we need it:
The
WithMetrics
method signature changed fromto
in #1223 but one occurrence was forgotten.
Which issue(s) this PR fixes
Fixes the following compilation error when
kube-state-metrics
is used as a GO dependency of an external project: