Merge pull request #858 from sebcdri/stopwaittime

Change `stop_wait_time` value to match Docker default

Author: LukasAud

README.md

@@ -406,7 +406,7 @@ docker::run { 'helloworld':
   after_start      => 'echo "container has started"',
   after            => [ 'container_b', 'mysql' ],
   depends          => [ 'container_a', 'postgres' ],
-  stop_wait_time   => 0,
+  stop_wait_time   => 10,
   read_only        => false,
   extra_parameters => [ '--restart=always' ],
 }

manifests/run.pp

@@ -244,7 +244,7 @@
   Boolean                                   $remove_container_on_stop          = true,
   Boolean                                   $remove_volume_on_start            = false,
   Boolean                                   $remove_volume_on_stop             = false,
-  Integer                                   $stop_wait_time                    = 0,
+  Integer                                   $stop_wait_time                    = 10,
   Optional[String]                          $syslog_identifier                 = undef,
   Optional[String]                          $syslog_facility                   = undef,
   Boolean                                   $read_only                         = false,

spec/defines/run_spec.rb

@@ -287,7 +287,7 @@
             'service_prefix'                    => 'docker-',
             'service_provider'                  => :undef,
             'socket_connect'                    => [],
-            'stop_wait_time'                    => 0,
+            'stop_wait_time'                    => 10,
             'syslog_identifier'                 => :undef,
             'systemd_restart'                   => 'on-failure',
             'tty'                               => false,