Merge lp:~andreserl/maas/lp1459865_1.8 into lp:maas/1.8

Proposed by Andres Rodriguez
Status: Merged
Approved by: Andres Rodriguez
Approved revision: no longer in the source branch.
Merged at revision: 4037
Proposed branch: lp:~andreserl/maas/lp1459865_1.8
Merge into: lp:maas/1.8
Diff against target: 58 lines (+25/-1)
3 files modified
docs/changelog.rst (+11/-0)
src/provisioningserver/kernel_opts.py (+1/-1)
src/provisioningserver/tests/test_kernel_opts.py (+13/-0)
To merge this branch: bzr merge lp:~andreserl/maas/lp1459865_1.8
Reviewer Review Type Date Requested Status
Andres Rodriguez (community) Approve
Review via email: mp+267878@code.launchpad.net

Commit message

Backport rev4182: Fix enlistment by ensuring correct kernel parameters are being passed & update changelog.

To post a comment you must log in.
Revision history for this message
Andres Rodriguez (andreserl) wrote :

self approve as this was approved in trunk and 1.8 is affected.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'docs/changelog.rst'
--- docs/changelog.rst 2015-08-05 15:57:00 +0000
+++ docs/changelog.rst 2015-08-12 21:24:01 +0000
@@ -3,6 +3,17 @@
3=========3=========
44
55
61.8.2
7=====
8
9See https://launchpad.net/maas/+milestone/1.8.2 for full details.
10
11Bug Fix Update
12--------------
13
14#1459865 Fix enlistment to always use the correct kernel parameters.
15
16
61.8.1171.8.1
7=====18=====
819
920
=== modified file 'src/provisioningserver/kernel_opts.py'
--- src/provisioningserver/kernel_opts.py 2015-07-13 21:00:00 +0000
+++ src/provisioningserver/kernel_opts.py 2015-08-12 21:24:01 +0000
@@ -127,7 +127,7 @@
127127
128def compose_purpose_opts(params):128def compose_purpose_opts(params):
129 """Return the list of the purpose-specific kernel options."""129 """Return the list of the purpose-specific kernel options."""
130 if params.purpose == "commissioning" or params.purpose == "xinstall":130 if params.purpose in ["commissioning", "xinstall", "enlist"]:
131 # These are kernel parameters read by the ephemeral environment.131 # These are kernel parameters read by the ephemeral environment.
132 tname = prefix_target_name(132 tname = prefix_target_name(
133 get_ephemeral_name(133 get_ephemeral_name(
134134
=== modified file 'src/provisioningserver/tests/test_kernel_opts.py'
--- src/provisioningserver/tests/test_kernel_opts.py 2015-07-13 21:00:00 +0000
+++ src/provisioningserver/tests/test_kernel_opts.py 2015-08-12 21:24:01 +0000
@@ -216,6 +216,19 @@
216 "overlayroot=tmpfs",216 "overlayroot=tmpfs",
217 "ip=::::%s:BOOTIF" % params.hostname]))217 "ip=::::%s:BOOTIF" % params.hostname]))
218218
219 def test_enlist_compose_kernel_command_line_inc_purpose_opts(self):
220 # The result of compose_kernel_command_line includes the purpose
221 # options for a non "commissioning" node.
222 params = self.make_kernel_parameters(purpose="enlist")
223 cmdline = compose_kernel_command_line(params)
224 self.assertThat(
225 cmdline,
226 ContainsAll([
227 "root=/dev/disk/by-path/ip-",
228 "iscsi_initiator=",
229 "overlayroot=tmpfs",
230 "ip=::::%s:BOOTIF" % params.hostname]))
231
219 def test_commissioning_compose_kernel_command_line_inc_extra_opts(self):232 def test_commissioning_compose_kernel_command_line_inc_extra_opts(self):
220 mock_get_curtin_sep = self.patch(233 mock_get_curtin_sep = self.patch(
221 kernel_opts, 'get_curtin_kernel_cmdline_sep')234 kernel_opts, 'get_curtin_kernel_cmdline_sep')

Subscribers

People subscribed via source and target branches