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

feat(): add security context on the container level #18

Open
wants to merge 1 commit into
base: release/5.0
Choose a base branch
from

Conversation

anja-istenic
Copy link

It is recommended practice to give a resource (pod/container) the least privileges it needs. The Lenses Helm Chart already supports configuring the security context on the pod level. This PR adds the securityContext configurations on the container level (more info on container level securityContext here).

This change is fully backwards compatible, as the containerSecurityContext value configuration is optional and defaults to {}

@anja-istenic
Copy link
Author

@georgeyord Not sure what the merge process is but any review or feedback would be much appreciated.

@waewoo
Copy link

waewoo commented Feb 16, 2023

Hello

I am also interested in this feature (internal security rules...).
When is it expected that this MR be merged ?

Thanks

PS:
It will be necessary to add it in lenses-provision container too.

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.

2 participants