launchpad-reviewers team mailing list archive

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

Raphaël Badin has proposed merging lp:~rvb/maas/qa-lab-tests-add-infos into lp:~maas-maintainers/maas/qa-lab-tests with lp:~rvb/maas/qa-lab-tests-port as a prerequisite.

Commit message:
Add debugging details.

Requested reviews:
  MAAS Maintainers (maas-maintainers)

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

=== modified file 'debian/tests/maas-integration.py'
--- debian/tests/maas-integration.py	2012-12-06 10:34:23 +0000
+++ debian/tests/maas-integration.py	2012-12-06 10:34:23 +0000
@@ -143,7 +143,10 @@
         return stdout, stderr
 
     def _run_maas_cli(self, args):
-        return self._run_command(["maas-cli", "maas"] + args)
+        result = self._run_command(["maas-cli", "maas"] + args)
+        self.addDetail(
+            'maas-cli maas %s' % str(args), text_content(str(result)))
+        return result
 
     def test_01_create_admin(self):
         """Run maas createsuperuser."""
@@ -357,6 +360,7 @@
 
     def get_juju_status(self):
         status_output = self._run_juju_command(["status"])
+        self.addDetail('juju status', text_content(str(status_output)))
         status = yaml.load(status_output)
         return status