Skip to content

Commit 8c5e99f

Browse files
p5thaJeztah
authored andcommitted
feat: manage docker group with systemd-sysusers
Switches away from the groupadd postinstall commands to managing the docker group with sysusers. This is a declarative way to create and manage users, better suited for the atomic distros such as Silverblue. Signed-off-by: Robert Sturla <[email protected]> Signed-off-by: Sebastiaan van Stijn <[email protected]>
1 parent 5b1a2b6 commit 8c5e99f

File tree

3 files changed

+7
-23
lines changed

3 files changed

+7
-23
lines changed

deb/common/docker-ce.postinst

Lines changed: 0 additions & 20 deletions
This file was deleted.

deb/common/rules

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -133,6 +133,9 @@ override_dh_auto_install:
133133
install -D -p -m 0755 engine/contrib/dockerd-rootless-setuptool.sh debian/docker-ce-rootless-extras/usr/bin/dockerd-rootless-setuptool.sh
134134
# TODO: how can we install vpnkit?
135135

136+
# install systemd sysusers config
137+
install -D -p -m 0644 engine/contrib/systemd-sysusers/docker.conf debian/docker-ce/usr/lib/sysusers.d/docker.conf
138+
136139
override_dh_installinit:
137140
# use "docker" as our service name, not "docker-ce"
138141
dh_installinit --name=docker

rpm/SPECS/docker-ce.spec

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -84,6 +84,9 @@ install -D -p -m 0755 $(readlink -f engine/bundles/dynbinary-daemon/dockerd) ${R
8484
install -D -p -m 0755 $(readlink -f engine/bundles/dynbinary-daemon/docker-proxy) ${RPM_BUILD_ROOT}%{_bindir}/docker-proxy
8585
install -D -p -m 0755 /usr/local/bin/docker-init ${RPM_BUILD_ROOT}%{_libexecdir}/docker/docker-init
8686

87+
# install systemd sysusers config
88+
install -D -p -m 0644 engine/contrib/systemd-sysusers/docker.conf ${RPM_BUILD_ROOT}%{_sysusersdir}/docker.conf
89+
8790
# install systemd scripts
8891
install -D -p -m 0644 engine/contrib/init/systemd/docker.service ${RPM_BUILD_ROOT}%{_unitdir}/docker.service
8992
install -D -p -m 0644 engine/contrib/init/systemd/docker.socket ${RPM_BUILD_ROOT}%{_unitdir}/docker.socket
@@ -100,14 +103,12 @@ mkdir -p ${RPM_BUILD_ROOT}/etc/docker
100103
%{_libexecdir}/docker/docker-init
101104
%{_unitdir}/docker.service
102105
%{_unitdir}/docker.socket
106+
%{_sysusersdir}/docker.conf
103107
%{_mandir}/man*/*
104108
%dir /etc/docker
105109

106110
%post
107111
%systemd_post docker.service
108-
if ! getent group docker > /dev/null; then
109-
groupadd --system docker
110-
fi
111112

112113
%preun
113114
%systemd_preun docker.service docker.socket

0 commit comments

Comments
 (0)