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
Prometheus lint errors in operator metrics #17789
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that in general we should conform to promtool
recommendations like this, so I'm fine with this from that perspective. If these are already in released versions of Cilium then we should also mention that the metrics are being renamed in the Upgrade guide. There should be examples like that already in the docs.
@krishgobinath could you add the information mentioned by Joe here in the upgrade guide? thank you |
4d9d320
to
d0e8b66
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good for me (pending Joe's comments are addressed, of course).
d0e8b66
to
fc83d31
Compare
Promtool identified following lint errors when running against operator metrics 1) cilium_operator_identity_gc_entries_total non-counter metrics should not have "_total" suffix 2) cilium_operator_identity_gc_runs_total non-counter metrics should not have "_total" suffix Add relevant changes in upgrade documentation for 1.10 and 1.11 Fixing both the non-counter metrics. Signed-off-by: Gobinath Krishnamoorthy <gobinathk@google.com>
fc83d31
to
98487cc
Compare
Promtool identified following lint errors when running against operator metrics
Fixing both the non-counter metrics.
Signed-off-by: Gobinath Krishnamoorthy gobinathk@google.com