Merge lp:~nobuto/maas-deployer/kernel-opts into lp:~maas-deployers/maas-deployer/next

Proposed by Nobuto Murata
Status: Merged
Merged at revision: 23
Proposed branch: lp:~nobuto/maas-deployer/kernel-opts
Merge into: lp:~maas-deployers/maas-deployer/next
Diff against target: 28 lines (+4/-2)
2 files modified
examples/deployment.yaml (+2/-0)
maas_deployer/vmaas/maasclient/clidriver.py (+2/-2)
To merge this branch: bzr merge lp:~nobuto/maas-deployer/kernel-opts
Reviewer Review Type Date Requested Status
MAAS Deployers Pending
Review via email: mp+268569@code.launchpad.net

Description of the change

maas-deployer already has ability to set kernel_opts. However it was not properly quoted.

quote value for maas set-config to handle commands like below,
LP: #1481688

$ maas maas maas set-config \
    name=kernel_opts value='console=tty0 console=ttyS1,115200n8'

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'examples/deployment.yaml'
--- examples/deployment.yaml 2015-08-11 08:55:09 +0000
+++ examples/deployment.yaml 2015-08-20 05:01:14 +0000
@@ -33,6 +33,8 @@
33 main_archive: http://archive.ubuntu.com/ubuntu33 main_archive: http://archive.ubuntu.com/ubuntu
34 upstream_dns: 192.168.122.134 upstream_dns: 192.168.122.1
35 maas_name: automaas35 maas_name: automaas
36 # kernel_opts: "console=tty0 console=ttyS1,115200n8"
37 # ntp_server: ntp.ubuntu.com
3638
37 # This section is used to define the networking parameters for when39 # This section is used to define the networking parameters for when
38 # the node first comes up. It is fed into the meta-data cloud-init40 # the node first comes up. It is fed into the meta-data cloud-init
3941
=== modified file 'maas_deployer/vmaas/maasclient/clidriver.py'
--- maas_deployer/vmaas/maasclient/clidriver.py 2015-08-11 08:55:09 +0000
+++ maas_deployer/vmaas/maasclient/clidriver.py 2015-08-20 05:01:14 +0000
@@ -51,9 +51,9 @@
51 value = kwargs[key]51 value = kwargs[key]
52 if isinstance(value, list):52 if isinstance(value, list):
53 for v in value:53 for v in value:
54 cmdarr.append("%s=%s" % (key, str(v)))54 cmdarr.append("%s='%s'" % (key, str(v)))
55 else:55 else:
56 cmdarr.append("%s=%s" % (key, str(value)))56 cmdarr.append("%s='%s'" % (key, str(value)))
57 stdout = execc(cmdarr)[0]57 stdout = execc(cmdarr)[0]
5858
59 display_stdout = stdout59 display_stdout = stdout

Subscribers

People subscribed via source and target branches

to all changes: