-
Notifications
You must be signed in to change notification settings - Fork 159
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
overlay/sysusers: remove 10-groups-basic.conf
which comes from setup
package
#2539
Conversation
`setup` package As `setup` shipped its own sysusers.d entries, see https://src.fedoraproject.org/rpms/setup/c/ff8169d809a5de20864deb6bb41c4b4104a3fb61?branch=f38 and https://src.fedoraproject.org/rpms/setup/c/9174b59bde57a991c284c10dac749d061f93aaad?branch=f38, we should remove the duplicated entries in `10-groups-basic.conf`. Xerf to coreos#1836
I've compared |
Thanks @travier for the checking, and |
Oh wow, I didn't realize we had another copy of that data! Nice work to delete it! |
OK now here's the tricky thing though...this content also gets pulled into openshift/os pretty quickly in the development stream, so anything that lands here generally will need to work on RHEL 9.2 as well. So the basic question is did systemd/systemd#24249 make it into RHEL 9.2 yet? |
(If it didn't, we can still land this PR here, but re-add the file in openshift/os) |
The overlay sysuser config files are under dir
Not yet, and sysusers configuration fragment for |
The "legacy" nobody group is enforced by: https://github.com/coreos/fedora-coreos-config/blob/testing-devel/overlay.d/15fcos/usr/lib/sysusers.d/00-coreos-nobody.conf |
As
setup
shipped its own sysusers.d entries,see https://src.fedoraproject.org/rpms/setup/c/ff8169d809a5de20864deb6bb41c4b4104a3fb61?branch=f38 and https://src.fedoraproject.org/rpms/setup/c/9174b59bde57a991c284c10dac749d061f93aaad?branch=f38, we should remove the duplicated entries in
10-groups-basic.conf
.Xerf to #1836