-
Notifications
You must be signed in to change notification settings - Fork 238
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
feat(serviceMonitors): add option of additional Labels in serviceMonitor #586
feat(serviceMonitors): add option of additional Labels in serviceMonitor #586
Conversation
Adding @VILJkid to add some thoughts on this as the original serviceMonitor feature contributor. Thanks. |
We will most likely merge the original backport 1.x PR before this one: |
Thanks @eyenx for this feature addition! |
Will do after lunch |
@VILJkid Done! |
e04e80b
to
1647306
Compare
Signed-off-by: Toni Tauro <[email protected]>
Signed-off-by: Toni Tauro <[email protected]>
Signed-off-by: Toni Tauro <[email protected]>
1647306
to
2d8e082
Compare
@peterzhuamazon rebased |
aaah my editor reformatted the whole CHANGELOG.. one sec. |
3eff7e2
to
16dcd2a
Compare
Signed-off-by: Toni Tauro <[email protected]>
16dcd2a
to
f468c04
Compare
Okay @peterzhuamazon @VILJkid fixed it. |
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 feel like this change is pretty straight forward.
What do you think about it @VILJkid @prudhvigodithi ?
Thanks.
LGTM, thanks @eyenx. |
Thanks @eyenx for the contribution and would you mind backport to 1.x as well? Thanks! |
I'll try to see if I can fit this into my schedule of this week. |
Description
Adds ability to add additional Labels to ServiceMonitor in case Prometheus is scraping for specific labels
Issues Resolved
#585
Check List
For any changes to files within Helm chart directories:
CHANGELOG.md
updated to reflect changeBy submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.