-
Notifications
You must be signed in to change notification settings - Fork 157
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
tests/ignition: check file ownership for system users #2670
base: testing-devel
Are you sure you want to change the base?
tests/ignition: check file ownership for system users #2670
Conversation
This ensure that entries in Ignition configuration can reference system users even if not present in ostree commit (e.g. `zincati). (cherry picked from commit dd25144)
Check the test is passed without
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Some context for #774: The use case where we need to run sysusers in the initramfs (#774) is when you want to declare a system user in a sysusers config file (written by Ignition to the real root) and also include a file in the Ignition config that is owned by that new system user. As the users will not be created in the real root when ignition-files runs, it would fail. So we need something like #774 to pre-create the users so that Ignition can set the ownership of the files it writes to the real root. This test looks fine to me but I'm not sure how much we already have testing that in kola. |
3af42d3
to
4ebeb00
Compare
Here is an example config that should fail:
See: https://www.freedesktop.org/software/systemd/man/sysusers.d.html |
This should fail with |
Use `dnsmasq` instead, as rhcos also includes this user.
4ebeb00
to
a474453
Compare
So the issue here is that the I commented in coreos/fedora-coreos-tracker#155 (comment) to try to keep design discussions there. |
Based on #774