-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
enhancement(helm platform): additional labels for Prometheus Operator #7474
enhancement(helm platform): additional labels for Prometheus Operator #7474
Conversation
nlamirault
commented
May 15, 2021
- enhancement(podmonitor.yaml and values.yaml): Added additional labels feature for PodMonitor for Prometheus Operator.
Signed-off-by: Nicolas Lamirault <nicolas.lamirault@gmail.com>
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.
Thanks @nlamirault definitely useful to allow - two suggested changes to the comments just for clarity.
@binarylogic should we add a |
Yes, I would add it here. That will sync it across everything. |
Link seems broken @binarylogic |
Sorry @nlamirault that was for me, private config repo - I'm adding a |
@nlamirault after running down some rabbit holes... https://github.com/timberio/vector/blob/master/.github/semantic.yml#L79 |
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.
LGTM once @spencergilbert's comment is resolved. 🙌
Signed-off-by: Nicolas Lamirault <nicolas.lamirault@gmail.com>