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

Helm chart is configured to run as root without ability to override #13869

Closed
megakoresh opened this issue Mar 30, 2021 · 3 comments · Fixed by #13917
Closed

Helm chart is configured to run as root without ability to override #13869

megakoresh opened this issue Mar 30, 2021 · 3 comments · Fixed by #13917

Comments

@megakoresh
Copy link
Contributor

megakoresh commented Mar 30, 2021

Helm is configured to run containers as root so that it can install packages. This is against security best practices and to top it all off, the chart does not allow editing this setting, and neither can the bootstrap be edited, so it doesn't crash when running as non-root. This makes the helm chart not suitable for secure production environments.

To fix this, it would be enough to allow editing the runAsUser field and the bootstrap script from values.yaml. Although in my opinion bad practices like this should not be promoted by the upstream, so it would be preferable if non-root was the default option, with documentation reflecting this.

I may have time to fix this later this week, but my schedule is quite tight so if someone has time to do this, please post here so I don't start duplicating someone's work.

@amitmiran137
Copy link
Member

@craig-rueda WDYT?

@craig-rueda
Copy link
Member

Yep, totally agree.

@megakoresh
Copy link
Contributor Author

Ok, working on it

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 a pull request may close this issue.

5 participants