-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Update node-exporter with PodSecurityPolicies #167
Update node-exporter with PodSecurityPolicies #167
Conversation
latchmihay
commented
Jul 24, 2019
- add pod security role to the node-exporter clusterRole
- add pod security policy for node-exporter
- add pod security policy for node-exporter
Thanks for all your awesome PRs! Because of how jsonnet-bundler works today, we need the jsonnet changes to be in one commit, and the bump of the jsonnet dependency in a separate commit (this is being fixed with jsonnet-bundler/jsonnet-bundler#36). Instructions how to do it now: https://github.com/coreos/kube-prometheus#contributing |
I wasn't too sure how that work when its coming from a fork and what the dependency versions actually updates to. When I tried I saw the versions changed on almost all of the dependencies in the lock file, and that didnt seem right to me. Can you provide me with step by step instructions I am happy to do it |
from the tip of this PR run
this will update the kube-prometheus dependency within this repo, and then you just commit that in a separate commit |
Can you rebase your PR one more time? We have finally gotten rid of the local dependency workaround and there shouldn't be another conflict with Make sure to |
I followed your instructions and recommited the latest |
@latchmihay you need to first rebase your PR to master upstream branch, then run |
- add pod security policy for node-exporter
thank you. I have rebased and regenerated |
I am closing this as PSPs are scheduled to be deprecated - kubernetes/kubernetes#97171 Please reopen or open an issue if you feel this is something that should be worked on. |