diff mbox series

[10/15] package/docker-engine: adapt to preset-all

Message ID 20191103165640.21819-11-jeremy.rosen@smile.fr
State Superseded
Headers show
Series use host-systemd to enable units | expand

Commit Message

Jérémy ROSEN Nov. 3, 2019, 4:56 p.m. UTC 
docker-engine has two services:
* docker.service, if enabled: docker will start at boot
* docker.socket, if enabled: docker will start at first connection

buildroot currently enables only the service
this patch keeps it that way.

Signed-off-by: Jérémy Rosen <jeremy.rosen@smile.fr>
---
 package/docker-engine/50-docker-engine.preset | 1 +
 package/docker-engine/docker-engine.mk        | 5 ++---
 2 files changed, 3 insertions(+), 3 deletions(-)
 create mode 100644 package/docker-engine/50-docker-engine.preset

Comments

James Hilliard Nov. 3, 2019, 5:25 p.m. UTC | #1 
On Sun, Nov 3, 2019 at 11:57 AM Jérémy Rosen <jeremy.rosen@smile.fr> wrote:
>
> docker-engine has two services:
> * docker.service, if enabled: docker will start at boot
> * docker.socket, if enabled: docker will start at first connection
>
> buildroot currently enables only the service
> this patch keeps it that way.
The socket not being enabled looks to be a bug so I don't think it
should be disabled.
>
> Signed-off-by: Jérémy Rosen <jeremy.rosen@smile.fr>
> ---
>  package/docker-engine/50-docker-engine.preset | 1 +
>  package/docker-engine/docker-engine.mk        | 5 ++---
>  2 files changed, 3 insertions(+), 3 deletions(-)
>  create mode 100644 package/docker-engine/50-docker-engine.preset
>
> diff --git a/package/docker-engine/50-docker-engine.preset b/package/docker-engine/50-docker-engine.preset
> new file mode 100644
> index 0000000000..0d8cf35345
> --- /dev/null
> +++ b/package/docker-engine/50-docker-engine.preset
> @@ -0,0 +1 @@
> +disable docker.socket
> diff --git a/package/docker-engine/docker-engine.mk b/package/docker-engine/docker-engine.mk
> index 6a225ee5f0..8540cda9af 100644
> --- a/package/docker-engine/docker-engine.mk
> +++ b/package/docker-engine/docker-engine.mk
> @@ -68,9 +68,8 @@ define DOCKER_ENGINE_INSTALL_INIT_SYSTEMD
>                 $(TARGET_DIR)/usr/lib/systemd/system/docker.service
>         $(INSTALL) -D -m 0644 $(@D)/contrib/init/systemd/docker.socket \
>                 $(TARGET_DIR)/usr/lib/systemd/system/docker.socket
> -       mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/
> -       ln -fs ../../../../usr/lib/systemd/system/docker.service \
> -               $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/docker.service
> +       $(INSTALL) -D -m 644 package/docker-engine/50-docker-engine.preset \
> +               $(TARGET_DIR)/usr/lib/systemd/system-preset/50-docker-engine.preset
>  endef
>
>  define DOCKER_ENGINE_INSTALL_INIT_SYSV
> --
> 2.24.0.rc1
>
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
Jérémy ROSEN Nov. 4, 2019, 5:03 p.m. UTC | #2 
In that case, it's open to discussion. Enabling both means that it's
started at boot, and the socket enablement does nothing.

Adding the socket means that it would automatically restart, once stopped
on connexions.

Enabling both is harmless, so I'll follow your advice

Le dim. 3 nov. 2019 à 18:25, James Hilliard <james.hilliard1@gmail.com> a
écrit :

> On Sun, Nov 3, 2019 at 11:57 AM Jérémy Rosen <jeremy.rosen@smile.fr>
> wrote:
> >
> > docker-engine has two services:
> > * docker.service, if enabled: docker will start at boot
> > * docker.socket, if enabled: docker will start at first connection
> >
> > buildroot currently enables only the service
> > this patch keeps it that way.
> The socket not being enabled looks to be a bug so I don't think it
> should be disabled.
> >
> > Signed-off-by: Jérémy Rosen <jeremy.rosen@smile.fr>
> > ---
> >  package/docker-engine/50-docker-engine.preset | 1 +
> >  package/docker-engine/docker-engine.mk        | 5 ++---
> >  2 files changed, 3 insertions(+), 3 deletions(-)
> >  create mode 100644 package/docker-engine/50-docker-engine.preset
> >
> > diff --git a/package/docker-engine/50-docker-engine.preset
> b/package/docker-engine/50-docker-engine.preset
> > new file mode 100644
> > index 0000000000..0d8cf35345
> > --- /dev/null
> > +++ b/package/docker-engine/50-docker-engine.preset
> > @@ -0,0 +1 @@
> > +disable docker.socket
> > diff --git a/package/docker-engine/docker-engine.mk
> b/package/docker-engine/docker-engine.mk
> > index 6a225ee5f0..8540cda9af 100644
> > --- a/package/docker-engine/docker-engine.mk
> > +++ b/package/docker-engine/docker-engine.mk
> > @@ -68,9 +68,8 @@ define DOCKER_ENGINE_INSTALL_INIT_SYSTEMD
> >                 $(TARGET_DIR)/usr/lib/systemd/system/docker.service
> >         $(INSTALL) -D -m 0644 $(@D)/contrib/init/systemd/docker.socket \
> >                 $(TARGET_DIR)/usr/lib/systemd/system/docker.socket
> > -       mkdir -p
> $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/
> > -       ln -fs ../../../../usr/lib/systemd/system/docker.service \
> > -
>  $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/docker.service
> > +       $(INSTALL) -D -m 644
> package/docker-engine/50-docker-engine.preset \
> > +
>  $(TARGET_DIR)/usr/lib/systemd/system-preset/50-docker-engine.preset
> >  endef
> >
> >  define DOCKER_ENGINE_INSTALL_INIT_SYSV
> > --
> > 2.24.0.rc1
> >
> > _______________________________________________
> > buildroot mailing list
> > buildroot@busybox.net
> > http://lists.busybox.net/mailman/listinfo/buildroot
>
diff mbox series

Patch

diff --git a/package/docker-engine/50-docker-engine.preset b/package/docker-engine/50-docker-engine.preset
new file mode 100644
index 0000000000..0d8cf35345
--- /dev/null
+++ b/package/docker-engine/50-docker-engine.preset
@@ -0,0 +1 @@ 
+disable docker.socket
diff --git a/package/docker-engine/docker-engine.mk b/package/docker-engine/docker-engine.mk
index 6a225ee5f0..8540cda9af 100644
--- a/package/docker-engine/docker-engine.mk
+++ b/package/docker-engine/docker-engine.mk
@@ -68,9 +68,8 @@  define DOCKER_ENGINE_INSTALL_INIT_SYSTEMD
 		$(TARGET_DIR)/usr/lib/systemd/system/docker.service
 	$(INSTALL) -D -m 0644 $(@D)/contrib/init/systemd/docker.socket \
 		$(TARGET_DIR)/usr/lib/systemd/system/docker.socket
-	mkdir -p $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/
-	ln -fs ../../../../usr/lib/systemd/system/docker.service \
-		$(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/docker.service
+	$(INSTALL) -D -m 644 package/docker-engine/50-docker-engine.preset \
+		$(TARGET_DIR)/usr/lib/systemd/system-preset/50-docker-engine.preset
 endef
 
 define DOCKER_ENGINE_INSTALL_INIT_SYSV