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

[kube-state-metrics] clusterRoleBinding and clusterRole name overrides #4548

Conversation

Maven35
Copy link

@Maven35 Maven35 commented May 18, 2024

What this PR does / why we need it

Which issue this PR fixes

(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

Special notes for your reviewer

Checklist

  • DCO signed
  • Chart Version bumped
  • Title of the PR starts with chart name (e.g. [prometheus-couchdb-exporter])

@Maven35 Maven35 requested review from dotdc, mrueg and tariq1890 as code owners May 18, 2024 23:15
@Maven35 Maven35 force-pushed the kube-state-metrics-clusterrole branch from cea8638 to a2521f0 Compare May 18, 2024 23:18
@Maven35
Copy link
Author

Maven35 commented May 20, 2024

ready for review @dotdc, @mrueg, @tariq1890 whenever you get a chance.

@Maven35
Copy link
Author

Maven35 commented May 31, 2024

anyone want to take a look at this? @dotdc, @mrueg, @tariq1890?

dotdc
dotdc previously approved these changes May 31, 2024
Copy link
Member

@dotdc dotdc left a comment

Choose a reason for hiding this comment

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

Sorry for the delay, lgtm.

@dotdc
Copy link
Member

dotdc commented May 31, 2024

@Maven35 Can you fix the trailing spaces?

charts/kube-state-metrics/values.yaml
  Error: 92:177 [trailing-spaces] trailing spaces

@dotdc
Copy link
Member

dotdc commented Jun 13, 2024

Hi @Maven35,
Do you plan to reopen this PR or create a new one?

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.

[kube-state-metrics] alter clusterRole and binding creation so that it accepts a custom name
2 participants