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

Merged
merged 8 commits into from
Nov 11, 2024

Conversation

scoof
Copy link
Contributor

@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
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 this and sorry for the delay in getting to it. LGTM

:shipit:

@skamboj skamboj force-pushed the improvement-metricrelabelings branch 2 times, most recently from 548d790 to 438c5d0 Compare November 11, 2024 14:11
@skamboj skamboj merged commit 02065cf into bloomberg:master Nov 11, 2024
3 checks passed
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