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] Allow adding volumeMounts and volumes in deployment #2296

Merged
merged 8 commits into from
Aug 2, 2022

Conversation

gadisn
Copy link
Contributor

@gadisn gadisn commented Jul 21, 2022

Signed-off-by: Gadi Sneh gadisn@gmail.com

What this PR does / why we need it:

Allow adding volumeMounts and volumes to the deployment.
For example, this can be used to mount a config map with definitions of metrics for CRD, see --custom-resource-state-config-file here.

Which issue this PR fixes

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

  • fixes #

Special notes for your reviewer:

Checklist

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

Signed-off-by: Gadi Sneh gadisn@gmail.com

Signed-off-by: gadisn <gadisn@users.noreply.github.com>
Signed-off-by: gadisn <gadisn@users.noreply.github.com>
Signed-off-by: gadisn <gadisn@users.noreply.github.com>
@gadisn
Copy link
Contributor Author

gadisn commented Jul 21, 2022

@dotdc the 'lint code base' and 'lint-test' voters doesn't run for some reason.
Seems to also be the case in prev' PRs.
Any idea what is the issue?
Thanks

charts/kube-state-metrics/Chart.yaml Outdated Show resolved Hide resolved
Signed-off-by: gadisn <gadisn@users.noreply.github.com>
@gadisn gadisn requested a review from monotek July 24, 2022 07:59
monotek
monotek previously approved these changes Jul 24, 2022
@gadisn
Copy link
Contributor Author

gadisn commented Jul 25, 2022

@dotdc can you please review this?

@gadisn
Copy link
Contributor Author

gadisn commented Jul 26, 2022

@mrueg , @tariq1890 - can you of you code owners please review?
Thanks.

@gadisn
Copy link
Contributor Author

gadisn commented Jul 31, 2022

@dotdc, @mrueg, @tariq1890 - can any of you code owners review this PR?
Thanks.

Signed-off-by: gadisn <gadisn@users.noreply.github.com>
Signed-off-by: gadisn <gadisn@users.noreply.github.com>
Signed-off-by: gadisn <gadisn@users.noreply.github.com>
Signed-off-by: gadisn <gadisn@users.noreply.github.com>
@gadisn gadisn requested a review from mrueg August 1, 2022 10:23
@mrueg mrueg merged commit b48194c into prometheus-community:main Aug 2, 2022
@mrueg
Copy link
Member

mrueg commented Aug 2, 2022

Thanks for your contribution!

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.

None yet

3 participants