Fix: requirements and dependencies

diff --git a/manifests/ce.pp b/manifests/ce.pp
index dc7eec48e20c8a912b9f24f2dbec57f1e6830fa2..25a2204a81efd96ac9caf8794be4fe98c7ab5ad1 100644 (file)
--- a/manifests/ce.pp
+++ b/manifests/ce.pp
@@ -1,4 +1,4 @@
-# See https://linuxconfig.org/how-to-install-docker-on-debian-9-stretch-linux
+# See https://docs.docker.com/engine/install/debian/
 class docker::ce inherits docker {
   $distro = downcase($::lsbdistid)
 
@@ -26,11 +26,21 @@ class docker::ce inherits docker {
     refreshonly => true,
   }
 
-  Package['docker'] {
-    name    => 'docker-ce',
+  # Docker community edition
+  package { 'docker-ce':
+    ensure  => present,
     require => [
       File['/etc/apt/sources.list.d/docker.list'],
       Exec['docker-apt-auto-update'],
     ],
   }
+
+  # Ensure that the distro package is absent
+  Package['docker'] {
+    ensure => absent,
+  }
+
+  Service['docker'] {
+    require => Package['docker-ce'],
+  }
 }

diff --git a/manifests/compose/upstream.pp b/manifests/compose/upstream.pp
index c1cd20c8249611f9070feebf467fa4ce771c8ea8..846b89dc56816c210d10907041a084228f6cbad4 100644 (file)
--- a/manifests/compose/upstream.pp
+++ b/manifests/compose/upstream.pp
@@ -1,3 +1,4 @@
+# See https://docs.docker.com/compose/install/
 class docker::compose::upstream(
     $version = '1.29.0',
   ) {
@@ -9,4 +10,8 @@ class docker::compose::upstream(
     command => "/usr/bin/curl -L \"${url}\" -o ${dest} && chmod +x ${dest}",
     creates => "${dest}",
   }
+
+  package { 'docker-compose':
+    ensure => absent,
+  }
 }

diff --git a/manifests/init.pp b/manifests/init.pp
index 85838af4fe585f73fa7a3ed338c65cbc2795d36a..9e08997f091b16616421499f9083a168f7230d07 100644 (file)
--- a/manifests/init.pp
+++ b/manifests/init.pp
@@ -4,7 +4,8 @@ class docker {
   }
 
   service { 'docker' :
-    ensure => running,
-    enable => true,
+    ensure  => running,
+    enable  => true,
+    require => Package['docker'],
   }
 }