Merge lp:~rvb/maas/packaging.bug-1083542 into lp:~maas-maintainers/maas/packaging

Proposed by Raphaël Badin
Status: Merged
Approved by: Raphaël Badin
Approved revision: no longer in the source branch.
Merged at revision: 152
Proposed branch: lp:~rvb/maas/packaging.bug-1083542
Merge into: lp:~maas-maintainers/maas/packaging
Diff against target: 25 lines (+4/-1)
2 files modified
debian/changelog (+3/-0)
debian/maas-cluster-controller.postinst (+1/-1)
To merge this branch: bzr merge lp:~rvb/maas/packaging.bug-1083542
Reviewer Review Type Date Requested Status
Gavin Panella (community) Approve
Review via email: mp+136380@code.launchpad.net

Commit message

Fix the name of the updated config file in debian/maas-cluster-controller.postinst (/etc/maas/pserv.yaml and not /etc/maas/pserv.conf).

To post a comment you must log in.
Revision history for this message
Raphaël Badin (rvb) wrote :

This needs to be backport to the quantal sru packaging branch.

Also, when the fix for bug 1081212 is backported to the precise sru packaging branch, this also needs to be backported to the precise sru packaging branch.

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 'debian/changelog'
2--- debian/changelog 2012-11-23 00:08:07 +0000
3+++ debian/changelog 2012-11-27 12:01:24 +0000
4@@ -1,6 +1,9 @@
5 maas (0.1+bzr1366+dfsg-0ubuntu2) UNRELEASED; urgency=low
6
7 [ Raphaël Badin ]
8+ * debian/maas-cluster-controller.postinst: Fix the name of the config
9+ file (/etc/maas/pserv.yaml and not /etc/maas/pserv.conf)
10+ (LP: #1083542).
11 * debian/extras/99-maas-sudoers: Add 'SETENV:' to sudo rule
12 to allow preserving the environment when running
13 /usr/sbin/maas-import-pxe-files via sudo.
14
15=== modified file 'debian/maas-cluster-controller.postinst'
16--- debian/maas-cluster-controller.postinst 2012-11-22 22:48:32 +0000
17+++ debian/maas-cluster-controller.postinst 2012-11-27 12:01:24 +0000
18@@ -59,7 +59,7 @@
19 HOSTPART=$(echo $RET|awk '{ split($0,array,"/")} END{print array[3] }')
20 # And substitute it in-place in pserv.conf on an indented, non-commented
21 # line.
22- sed -ri "s|^([[:space:]]+generator:[[:space:]]+https?://)[^:/]+|\1$HOSTPART|" /etc/maas/pserv.conf
23+ sed -ri "s|^([[:space:]]+generator:[[:space:]]+https?://)[^:/]+|\1$HOSTPART|" /etc/maas/pserv.yaml
24 fi
25 fi
26

Subscribers

People subscribed via source and target branches