Skip to content
This repository has been archived by the owner on Jul 5, 2023. It is now read-only.

Add better docs for psp clusters #68

Merged
merged 5 commits into from
Sep 11, 2019

Conversation

johscheuer
Copy link
Contributor

Document what is required in order to run illuminatio in restricted clusters with psp.

See: #67

@johscheuer johscheuer added the documentation Improvements or additions to documentation label Sep 10, 2019
Copy link
Member

@maxbischoff maxbischoff left a comment

Choose a reason for hiding this comment

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

Some small details, otherwise lgtm.

Should we simply provide a PodSecurityPolicy that contains these? Might be simpler than maintaining documentation in the long run. (I would still merge this and address it in a later PR if you agree)

README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
@johscheuer
Copy link
Contributor Author

I updated the docs and fixed the docker cli image to use a different user than root (and some other minor issues)

@johscheuer
Copy link
Contributor Author

I created an issue for providing a PSP -> #73

IMHO if you run a Cluster with PSP activated you should be able to write it yourself since most tooling in the Kubernetres ecosystem probably doesn't provide them (which doesn't mean we shouldn't :))

Copy link
Member

@maxbischoff maxbischoff left a comment

Choose a reason for hiding this comment

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

Minor changes again, thanks

illuminatio-runner.dockerfile Show resolved Hide resolved
Dockerfile Show resolved Hide resolved
@johscheuer
Copy link
Contributor Author

@maxbischoff so good you need to approve twice ! 😂

@maxbischoff maxbischoff merged commit f9276c3 into inovex:master Sep 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants