-
Notifications
You must be signed in to change notification settings - Fork 167
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
Mount path /tmp as emptyDir volume #3150
Labels
area/charts
good first issue
Good issue for a new contributor to handle
help-wanted
Community help on this would be appreciated
kind/enhancement
priority/normal
Milestone
Comments
I've been thinking about this for a while as well, albeit for different reasons. emptyDir seems safe and easy. Mounting any other sort of volume introduces security implications that I'd rather not grapple with at this point in time. |
krancour
added
needs/area
good first issue
Good issue for a new contributor to handle
help-wanted
Community help on this would be appreciated
and removed
needs/priority
needs/area
labels
Dec 17, 2024
@SD-13 it's yours. |
Hey @SD-13 you workin' on this, if not could i take it ? |
@RohanMishra315 , I am working on it. Please pick some other issues |
@ggogel, No problem! |
3 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
area/charts
good first issue
Good issue for a new contributor to handle
help-wanted
Community help on this would be appreciated
kind/enhancement
priority/normal
Checklist
kargo version
, if applicable.Proposed Feature
The path
/tmp
is used in the promotion step git clone to save the cloned git repository. Currently, this path is part of the container's root filesystem. I'm proposing to mount an ephemeral emptyDir volume to this path.Motivation
We are about to roll out Kargo to our production stage. One of the compliance requirements is to enable the
securityContext
readOnlyRootFilesystem
. This protects the root filesystem of the container from being changed during runtime and therefore increases security. When we enable this, the promotion step git clone fails with the following error:Suggested Implementation
I see two possible implementation options in the Kargo Helm Chart:
/tmp
to the container by default.The text was updated successfully, but these errors were encountered: