launchpad-reviewers team mailing list archive

[Raphaël Badin <raphael.badin@xxxxxxxxxxxxx>]

Raphaël Badin has proposed merging lp:~rvb/maas/qa-lab-tests-slow-things-down into lp:~maas-maintainers/maas/qa-lab-tests.

Commit message:
Wait a bit between tests to work around bug 1171418.

Requested reviews:
  MAAS Maintainers (maas-maintainers)

For more details, see:
https://code.launchpad.net/~rvb/maas/qa-lab-tests-slow-things-down/+merge/160467
-- 
https://code.launchpad.net/~rvb/maas/qa-lab-tests-slow-things-down/+merge/160467
Your team MAAS Maintainers is requested to review the proposed merge of lp:~rvb/maas/qa-lab-tests-slow-things-down into lp:~maas-maintainers/maas/qa-lab-tests.

=== modified file 'maas-integration.py'
--- maas-integration.py	2013-04-01 18:19:28 +0000
+++ maas-integration.py	2013-04-23 18:33:02 +0000
@@ -507,6 +507,9 @@
                     "power_parameters_power_pass=%s" % power_pass])
 
     def test_commission_nodes(self):
+        # Wait a bit to let all the nodes go down.
+        # XXX: rvb 2013-04-23 bug=1171418
+        sleep(15)
         # Use maas-cli to accept all nodes.
         output, err = self._run_maas_cli(["nodes","accept-all"])
         for node in loads(output):
@@ -554,6 +557,9 @@
     @skipIf(DO_NOT_TEST_JUJU, "Not testing juju")
     @timeout(50*60)
     def test_juju_bootstrap(self):
+        # Wait a bit to let all the nodes go down.
+        # XXX: rvb 2013-04-23 bug=1171418
+        sleep(15)
         self.setup_juju()
         output = self._run_juju_command(['bootstrap'])
         self.addDetail('juju bootstrap', text_content(str(output)))