puppetlabs#627 and puppetlabs#526: Fix problems with image version "l…

…atest"

If image version is set to "latest"

- there are changes on each run, even if nothing is really done
- docker containers are restarted every time

This is fixed here.

However, this needs a change in using docker::run, which now needs an additional subscribe => Docker::Image[...].

See puppetlabs#627

manifests/image.pp

@@ -147,6 +147,7 @@
       logoutput   => true,
     }
   } elsif $ensure == 'latest' or $image_tag == 'latest' or $force {
+    /*
     notify { "Check if image ${image_arg} is in-sync":
       noop => false,
     }
@@ -159,14 +160,15 @@
       provider    => $exec_provider,
       logoutput   => true,
     }
-    ~> exec { "echo 'Update of ${image_arg} complete'":
+    ~> */ exec { "echo 'Update of ${image_arg} complete'":
       environment => $exec_environment,
       path        => $exec_path,
       timeout     => $exec_timeout,
       require     => File[$update_docker_image_path],
       provider    => $exec_provider,
       logoutput   => true,
-      refreshonly => true,
+      #refreshonly => true,
+      onlyif      => $image_install,
     }
   } elsif $ensure == 'present' {
     exec { $image_install: