Add retries for non transient file transfer errors

W/o this patch, Rabbitmq::Install::Rabbitmqadmin/Staging::File[rabbitmqadmin] /Exec[/var/lib/rabbitmq/rabbitmqadmin] sometimes could fail the curl command due to transient or connectivity errors. The solution is: - For transient errors: * use timeout param for staging::file * use --retries , --retry-delay flag for curl command passed to staging::file - For connectivity errors: * improve staging module to retry its commands as well, see https://github.com/nanliu/puppet-staging/pull/52 Closes-bug: #MODULES-1650 Signed-off-by: Bogdan Dobrelya <[email protected]>
jguiditta · Jan 13, 2015 · 966e0b5 · 966e0b5
1 parent cf30101
commit 966e0b5
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/manifests/install/rabbitmqadmin.pp b/manifests/install/rabbitmqadmin.pp
@@ -15,7 +15,8 @@
   staging::file { 'rabbitmqadmin':
     target      => '/var/lib/rabbitmq/rabbitmqadmin',
     source      => "${protocol}://${default_user}:${default_pass}@localhost:${management_port}/cli/rabbitmqadmin",
-    curl_option => '-k --noproxy localhost',
+    curl_option => '-k --noproxy localhost --retry 30 --retry-delay 6',
+    timeout     => '180',
     wget_option => '--no-proxy',
     require     => [
       Class['rabbitmq::service'],