-
Notifications
You must be signed in to change notification settings - Fork 5.3k
[kube-state-metrics]: add option to set user namespaces #6024
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
base: main
Are you sure you want to change the base?
Conversation
zanhsieh
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@KyriosGN0 Can you bump the chart version plz?
@zanhsieh i did bump the chart's version to 6.5.0 |
|
hey @dacamposol, could a i get a review here? i have addressed your review |
Head branch was pushed to by a user without write access
aedfa1c to
9ae94e8
Compare
|
hey @dotdc @tariq1890 , gentle ping on this PR |
Signed-off-by: AvivGuiser <[email protected]>
…ommunity#6048) Signed-off-by: yellowhat <[email protected]> Signed-off-by: AvivGuiser <[email protected]>
Signed-off-by: AvivGuiser <[email protected]>
Signed-off-by: AvivGuiser <[email protected]>
Signed-off-by: AvivGuiser <[email protected]>
Signed-off-by: AvivGuiser <[email protected]>
Signed-off-by: AvivGuiser <[email protected]>
0b50ef8 to
8d94fb8
Compare
Signed-off-by: AvivGuiser [email protected]
What this PR does / why we need it
Which issue this PR fixes
This PR intorduces the option to use user namespaces in k8s version 1.33 and higher, by default it is unset.
(optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)format, will close that issue when PR gets merged)Special notes for your reviewer
Checklist
[prometheus-couchdb-exporter])