Skip to content

Fixes suspected duplicate label panic for some GCP metric#153

Merged
SuperQ merged 2 commits intoprometheus-community:masterfrom
gidesh:master
Apr 22, 2022
Merged

Fixes suspected duplicate label panic for some GCP metric#153
SuperQ merged 2 commits intoprometheus-community:masterfrom
gidesh:master

Conversation

@gidesh
Copy link
Contributor

@gidesh gidesh commented Apr 20, 2022

Metric like istio.io/control were failing because mesh_id and some other labels were added multiple times

@gidesh
Copy link
Contributor Author

gidesh commented Apr 20, 2022

@SuperQ would be great if you can review it if possible as a fix for #85. We need this to be in prod as soon as possible.

Metric like `istio.io/control` were failing because `mesh_id` and some other labels were added multiple times

Signed-off-by: Gidesh Pampingal <gidesh.pampingal@johnlewis.co.uk>
As suggested in PR

Signed-off-by: Gidesh Pampingal <gidesh.pampingal@johnlewis.co.uk>
Copy link
Contributor

@SuperQ SuperQ left a comment

Choose a reason for hiding this comment

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

LGTM

@gidesh
Copy link
Contributor Author

gidesh commented Apr 21, 2022

Thanks @SuperQ . Are you ok to merge it and create a new release tag ?

@SuperQ
Copy link
Contributor

SuperQ commented Apr 22, 2022

The CI lint issue is fixed in master, going to merge this.

@SuperQ SuperQ merged commit df5aae7 into prometheus-community:master Apr 22, 2022
@svenmueller
Copy link

There is no new release containing this change yet. Any time line for it?

@kgeckhart kgeckhart mentioned this pull request Jan 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants