-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Umask=xxxx option to replace PodmanArgs=--umask=xxxx #25278
Comments
would you like to open a PR to add it? |
Yes, though I have to admit I have not yet written Go. So I am trying to learn it a bit before I open one. I apologise if it's not ok to submit a feature request with no PR to back it up. |
@geraveoyomama That is fine, adding a new field is pretty straight forward once you know what you have to touch, i.e. here is an example commit 54fce37 |
Please add Mask as well if you are going to add Umask. |
There is no --mask option? And not sure what that would be. Only --umask exist to set the process umask. |
looking through the docs for quadlets I do already see a |
Oops my brain was not reading the correctl. I saw umask and I was thinking Unmask/Mask for hiding parts of /sys and /proc/sys. Never mind. |
Feature request description
Adding a
Umask=
option to the quadlet generator to replace umask setting by way ofPodmanArgs=--umask=xxxx
. Since --umask is an explicit option and not a kit-bashed solution I think its deserving of its own generator key.Suggest potential solution
Add
Umask=xxxx
to the generator in lieu ofPodmanArgs=--umask=xxxx
.Have you considered any alternatives?
continue to use
PodmanArgs=--umask=xxxx
but its not so pretty nor set-and-forget friendly.Additional context
No response
The text was updated successfully, but these errors were encountered: