Skip to content
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

improvement: support relabelings in ServiceMonitor #148

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

scoof
Copy link

@scoof scoof commented Aug 12, 2024

Describe your changes
Added optional support for metricRelabelings

Testing performed
ServiceMonitor renders cleanly with or without relabelings

@hcsaustrup
Copy link

Prioritize this, please.

@scoof scoof force-pushed the improvement-metricrelabelings branch from 0c26b59 to b07803d Compare August 26, 2024 10:24
Signed-off-by: Sachin Kamboj <[email protected]>
Copy link
Contributor

@skamboj skamboj left a comment

Choose a reason for hiding this comment

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

Thank you so much for your contribution and apologies for the delay in getting to this.

I have a small comment about changing the default type of metricsRelabelings but other than that, this looks good.

Signed-off-by: Andreas Plesner <[email protected]>
@scoof scoof requested a review from skamboj November 4, 2024 11:10
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.

3 participants