Merge lp:~jtv/maas-test/maasfixture-log-earlier into lp:maas-test

Proposed by Jeroen T. Vermeulen
Status: Merged
Approved by: Jeroen T. Vermeulen
Approved revision: 144
Merged at revision: 144
Proposed branch: lp:~jtv/maas-test/maasfixture-log-earlier
Merge into: lp:maas-test
Diff against target: 30 lines (+9/-4)
1 file modified
maastest/maasfixture.py (+9/-4)
To merge this branch: bzr merge lp:~jtv/maas-test/maasfixture-log-earlier
Reviewer Review Type Date Requested Status
Gavin Panella (community) Approve
Raphaël Badin (community) Approve
Review via email: mp+214718@code.launchpad.net

Commit message

Request gathering of MAAS-side logs from an earlier stage in MAASFixture setup. The CI lab is hitting errors while making its first API request during MAASFixture setup, and this should add useful debugging information to that failure.

To post a comment you must log in.
Revision history for this message
Raphaël Badin (rvb) :
review: Approve
Revision history for this message
Gavin Panella (allenap) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'maastest/maasfixture.py'
2--- maastest/maasfixture.py 2014-04-03 02:10:29 +0000
3+++ maastest/maasfixture.py 2014-04-08 10:58:53 +0000
4@@ -281,6 +281,15 @@
5 # MAAS setup.
6 if self.proxy_url: # None or '' indicates no proxy.
7 self.configure_http_proxy(self.proxy_url)
8+
9+ # We should have a working MAAS at this point, so if things break from
10+ # here on, try to provide relevant information for debugging.
11+ self.log_connection_details()
12+ self.addCleanup(self.collect_logs)
13+ self.addCleanup(self.dump_data)
14+
15+ # Now do the rest of the setup: import images, configure the
16+ # controllers, and so on.
17 self.import_maas_images(
18 series=self.series, architecture=self.architecture,
19 simplestreams_filter=self.simplestreams_filter)
20@@ -288,10 +297,6 @@
21 if self.kvm_fixture.direct_ip is not None:
22 self.check_cluster_connected()
23 self.configure_cluster()
24- # Cleanups registration.
25- self.addCleanup(self.collect_logs)
26- self.addCleanup(self.dump_data)
27- self.log_connection_details()
28
29 def log_connection_details(self):
30 """Log the details on how to connect to this MAAS server."""

Subscribers

People subscribed via source and target branches