diff mbox

[OSSTEST,2/9] mg-schema-test-database: Provide some timeouts which are better for testing

Message ID 1450371968-27997-2-git-send-email-ian.jackson@eu.citrix.com (mailing list archive)
State New, archived
Headers show

Commit Message

Ian Jackson Dec. 17, 2015, 5:06 p.m. UTC
The default timeouts mean that after starting a test db queue daemon
and a test db allocation attempt, we have to wait two minutes.

Lower timeouts increase the risk that we might lose noncritical races
and allocate resources to the `wrong' tasks.  And they reduce the
duration of an outage which will cause a planned allocation attempt to
fail.

I think we don't care about those problems for test instances.

Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
---
 mg-schema-test-database |    2 ++
 1 file changed, 2 insertions(+)

Comments

Ian Campbell Dec. 17, 2015, 5:19 p.m. UTC | #1
On Thu, 2015-12-17 at 17:06 +0000, Ian Jackson wrote:
> The default timeouts mean that after starting a test db queue daemon
> and a test db allocation attempt, we have to wait two minutes.
> 
> Lower timeouts increase the risk that we might lose noncritical races
> and allocate resources to the `wrong' tasks.  And they reduce the
> duration of an outage which will cause a planned allocation attempt to
> fail.
> 
> I think we don't care about those problems for test instances.
> 
> Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>

Acked-by: Ian Campbell <ian.campbell@citrix.com>
diff mbox

Patch

diff --git a/mg-schema-test-database b/mg-schema-test-database
index bf82c75..818cf89 100755
--- a/mg-schema-test-database
+++ b/mg-schema-test-database
@@ -336,6 +336,8 @@  QueueDaemonHost $ctrlhost
 OwnerDaemonPort ${ctrlports%,*}
 QueueDaemonPort ${ctrlports#*,}
 ExecutiveDbOwningRoleRegexp .*
+QueueDaemonHoldoff 3
+QueueDaemonRetry 5
 END
 	mv -f $tcfg.tmp $tcfg