Skip to content

Conversation

@justin0u0
Copy link
Contributor

Hi 👋,

I would like introduces an enhancement to the Helm chart by allowing users to configure the spec.endpoints[].relabelings field in the ServiceMonitor. This will enable the export of additional metrics (e.g. the Node name of the Pod) without requiring manual modifications to the chart.

@coveralls
Copy link

coveralls commented Oct 1, 2024

Pull Request Test Coverage Report for Build 11118372036

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 62.259%

Totals Coverage Status
Change from base Build 11102737348: 0.0%
Covered Lines: 452
Relevant Lines: 726

💛 - Coveralls

@zbindenren
Copy link
Member

Thx for the MR. Could you document the additional parameter in the Readme?

@justin0u0
Copy link
Contributor Author

Thx for the MR. Could you document the additional parameter in the Readme?

Sure, I've documented the additional parameter in the README in commit 2a107b0.

Thanks for the reminder!

@zbindenren zbindenren merged commit cd56ef0 into postfinance:main Oct 1, 2024
5 checks passed
@justin0u0 justin0u0 deleted the support-service-monitor-relabelings branch October 2, 2024 05:51
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