From a665f7b9709fd3bca1ebfc7b7bd8e8dd4cae8a28 Mon Sep 17 00:00:00 2001 From: Matthew Treinish Date: Fri, 13 Mar 2015 16:14:53 -0400 Subject: [PATCH] Set timeouts for db upgrade operations This commit is adding timeout fields to the 2 execs as part of a new subunit2sql release. The first sets the timeout for the mysqldump to be 30min which should be sufficient time for that. The timeout for the actual db upgrade command is disabled however, because there is no way to safely estimate how long the migration will take. Change-Id: I7deb7abca9c97186343eb6c3ea30185c23c24c7a --- manifests/server.pp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/manifests/server.pp b/manifests/server.pp index 857b295..6169770 100644 --- a/manifests/server.pp +++ b/manifests/server.pp @@ -46,6 +46,7 @@ class subunit2sql::server ( subscribe => Package['subunit2sql'], require => File['/etc/subunit2sql-my.cnf'], refreshonly => true, + timeout => 1800, } exec { 'upgrade_subunit2sql_db': @@ -53,5 +54,6 @@ class subunit2sql::server ( path => '/usr/local/bin:/usr/bin:/bin/', subscribe => Exec['backup_subunit2sql_db'], refreshonly => true, + timeout => 0, } }