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

Update node-exporter with PodSecurityPolicies #167

Closed

Conversation

latchmihay
Copy link
Contributor

  • add pod security role to the node-exporter clusterRole
  • add pod security policy for node-exporter

- add pod security policy for node-exporter
@brancz
Copy link
Collaborator

brancz commented Jul 24, 2019

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

@latchmihay
Copy link
Contributor Author

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

@brancz
Copy link
Collaborator

brancz commented Jul 24, 2019

from the tip of this PR run

jb update

this will update the kube-prometheus dependency within this repo, and then you just commit that in a separate commit

@metalmatze
Copy link
Member

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 jsonnetfile.lock.json again.

Make sure to go get -u github.com/jsonnet-bundler/jsonnet-bundler/cmd/jb to install the latest version of jb for a new needed feature.

@latchmihay
Copy link
Contributor Author

I followed your instructions and recommited the latest jsonetfile.lock.json but there are still conflicts.
Not sure how I should resolve

@lilic
Copy link
Contributor

lilic commented Aug 30, 2019

@latchmihay you need to first rebase your PR to master upstream branch, then run jb update in the jsonnet/kube-prometheus directory and then run make generate-in-docker from the root dir.

@latchmihay
Copy link
Contributor Author

thank you. I have rebased and regenerated

Base automatically changed from master to main February 24, 2021 13:51
@paulfantom
Copy link
Member

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.

@paulfantom paulfantom closed this Feb 25, 2021
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.

5 participants