Umask=xxxx option to replace PodmanArgs=--umask=xxxx

[geraveoyomama]

Feature request description

Adding a Umask= option to the quadlet generator to replace umask setting by way of PodmanArgs=--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 of PodmanArgs=--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

[giuseppe]

would you like to open a PR to add it?

[geraveoyomama]

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.

[Luap99]

@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

[rhatdan]

Please add Mask as well if you are going to add Umask.

[Luap99]

There is no --mask option? And not sure what that would be. Only --umask exist to set the process umask.

[geraveoyomama]

looking through the docs for quadlets I do already see a Mask= option.

[rhatdan]

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.

[geraveoyomama]

opened a new PR because I didn't anticipate a repo sync would kill the PR.

It does say that #25369 is failing in root host sqlite but I do not know what to do about that.