Merge lp:~doanac/utah/dev+trunk into lp:utah

Proposed by Andy Doan
Status: Merged
Merged at revision: 845
Proposed branch: lp:~doanac/utah/dev+trunk
Merge into: lp:utah
Diff against target: 50 lines (+20/-5)
2 files modified
debian/changelog (+13/-1)
utah/provisioning/baremetal/cobbler.py (+7/-4)
To merge this branch: bzr merge lp:~doanac/utah/dev+trunk
Reviewer Review Type Date Requested Status
Max Brustkern (community) Approve
Review via email: mp+156057@code.launchpad.net

Description of the change

this gets our latest fixes from trunk into dev so that we can make this our new permanent dev focus.

To post a comment you must log in.
Revision history for this message
Max Brustkern (nuclearbob) wrote :

Seems reasonable to me.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2013-03-27 12:01:13 +0000
3+++ debian/changelog 2013-03-28 19:48:23 +0000
4@@ -4,7 +4,19 @@
5
6 -- Javier Collado <javier.collado@canonical.com> Wed, 27 Mar 2013 13:00:47 +0100
7
8-utah (0.9ubuntu2) quantal; urgency=low
9+utah (0.9.2ubuntu1) precise; urgency=low
10+
11+ * Moving removenfs to avoid a race condition on some machines
12+
13+ -- Max Brustkern <max@canonical.com> Wed, 27 Mar 2013 11:33:20 -0400
14+
15+utah (0.9.1ubuntu1) UNRELEASED; urgency=low
16+ [ Andy Doan ]
17+ * remove desktop check of wait_for_booted (LP: #1160148)
18+
19+ -- Andy Doan <andy.doan@canonical.com> Tue, 26 Mar 2013 13:38:24 -0500
20+
21+utah (0.9ubuntu2) UNRELEASED; urgency=low
22
23 [ Javier Collado ]
24 * Added battery_measurements boolean variable to runlist (defaults to False)
25
26=== modified file 'utah/provisioning/baremetal/cobbler.py'
27--- utah/provisioning/baremetal/cobbler.py 2013-03-21 14:50:49 +0000
28+++ utah/provisioning/baremetal/cobbler.py 2013-03-28 19:48:23 +0000
29@@ -196,14 +196,17 @@
30 self.rsyslog.wait_for_install(config.install_steps,
31 self._disable_netboot)
32
33- if self.installtype == 'desktop':
34- # TODO: look into getting this working for d-i installs
35- self.rsyslog.wait_for_booted(config.boot_steps)
36- self._removenfs()
37+ # TODO: wait_for_booted isn't working on d-i installs and
38+ # autopilot jobs like:
39+ # http://10.97.0.1:8080/view/autopilot/job/ps-unity-autopilot-release-testing/140/label=autopilot-nvidia/
40+ # self.rsyslog.wait_for_booted(config.boot_steps)
41
42 retry(self.sshcheck, logmethod=self.logger.info,
43 retry_timeout=config.checktimeout)
44
45+ if self.installtype == 'desktop':
46+ self._removenfs()
47+
48 self.provisioned = True
49 self.active = True
50 self._uuid_check()

Subscribers

People subscribed via source and target branches

to all changes: