Merge lp:~allenap/maas/packaging.fix-patch-02 into lp:~maas-maintainers/maas/packaging

Proposed by Gavin Panella
Status: Merged
Approved by: Gavin Panella
Approved revision: 340
Merged at revision: 341
Proposed branch: lp:~allenap/maas/packaging.fix-patch-02
Merge into: lp:~maas-maintainers/maas/packaging
Diff against target: 41 lines (+6/-13)
1 file modified
debian/patches/02-pserv-config.patch (+6/-13)
To merge this branch: bzr merge lp:~allenap/maas/packaging.fix-patch-02
Reviewer Review Type Date Requested Status
Andres Rodriguez (community) Approve
Raphaël Badin (community) Approve
Review via email: mp+246544@code.launchpad.net

Commit message

Fixes a patch that no longer applies cleanly because the OOPS section in the config file was removed.

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

It looks good: it applies cleanly now.

Now, I'm told it cannot be avoided but it's painful to get these silly questions about config files I didn't touch when upgrading: http://paste.ubuntu.com/9754896/

I'm +1 on this (since it fixes my immediate problem). But maybe it's worth Andres having a look at this.

(Also, IIRC you need a changelog entry for this)

review: Approve
Revision history for this message
Gavin Panella (allenap) wrote :

Thanks. I'll wait for roaksoax to comment too before I land.

Revision history for this message
Andres Rodriguez (andreserl) :
review: Approve
Revision history for this message
Raphaël Badin (rvb) wrote :

@Andres: don't we need a changelog entry for this?

Revision history for this message
Gavin Panella (allenap) wrote :

I don't think it needs a changelog entry. It's just a repair, not a change that anyone's going want to read about.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/patches/02-pserv-config.patch'
--- debian/patches/02-pserv-config.patch 2014-03-28 06:50:05 +0000
+++ debian/patches/02-pserv-config.patch 2015-01-15 10:09:46 +0000
@@ -3,8 +3,9 @@
3 include port, logfile, oops directory, cobbler url and username3 include port, logfile, oops directory, cobbler url and username
4Author: Andres Rodriguez <andreserl@ubuntu.com>4Author: Andres Rodriguez <andreserl@ubuntu.com>
55
6--- a/etc/maas/pserv.yaml6=== modified file 'etc/maas/pserv.yaml'
7+++ b/etc/maas/pserv.yaml7--- a/etc/maas/pserv.yaml 2015-01-14 16:24:44 +0000
8+++ b/etc/maas/pserv.yaml 2015-01-15 10:05:50 +0000
8@@ -6,7 +6,7 @@9@@ -6,7 +6,7 @@
9 ## running server.10 ## running server.
10 #11 #
@@ -12,18 +13,9 @@
12-logfile: "/dev/null"13-logfile: "/dev/null"
13+logfile: "/var/log/maas/pserv.log"14+logfile: "/var/log/maas/pserv.log"
14 15
15 ## OOPS configuration (optional).16 ## TFTP configuration.
16 #17 #
17@@ -15,7 +15,7 @@18@@ -18,8 +18,6 @@
18 # or directories other than what the oops machinery creates there.
19 #
20 # directory:
21- directory: "logs/oops"
22+ directory: "/var/log/maas/oops"
23 # reporter:
24 reporter: "maas-pserv"
25
26@@ -38,8 +38,6 @@
27 # resource_root: /var/lib/maas/boot-resources/current/19 # resource_root: /var/lib/maas/boot-resources/current/
28 20
29 # port: 6921 # port: 69
@@ -32,3 +24,4 @@
32 # generator: http://localhost/MAAS/api/1.0/pxeconfig/24 # generator: http://localhost/MAAS/api/1.0/pxeconfig/
33- generator: http://localhost:5243/api/1.0/pxeconfig/25- generator: http://localhost:5243/api/1.0/pxeconfig/
34 26
27

Subscribers

People subscribed via source and target branches