Merge lp:~andreserl/maas/packaging_fix_clusterd_logging into lp:~maas-maintainers/maas/packaging

Proposed by Andres Rodriguez
Status: Merged
Approved by: Andres Rodriguez
Approved revision: 348
Merged at revision: 348
Proposed branch: lp:~andreserl/maas/packaging_fix_clusterd_logging
Merge into: lp:~maas-maintainers/maas/packaging
Diff against target: 81 lines (+20/-16)
4 files modified
debian/changelog (+1/-1)
debian/maas-cluster-controller.preinst (+6/-0)
debian/patches/01-fix-database-settings.patch (+9/-3)
debian/patches/02-pserv-config.patch (+4/-12)
To merge this branch: bzr merge lp:~andreserl/maas/packaging_fix_clusterd_logging
Reviewer Review Type Date Requested Status
Gavin Panella (community) Approve
Andres Rodriguez (community) Approve
Review via email: mp+247360@code.launchpad.net

Commit message

Fix logging to remove pserv.log and not specify logging file in pserv.yaml

To post a comment you must log in.
Revision history for this message
Gavin Panella (allenap) wrote :

My packaging change had a companion branch for trunk which landed after. I think this change was made in the interval between the two branches.

review: Needs Fixing
Revision history for this message
Andres Rodriguez (andreserl) wrote :

Fixed. Self approving.

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

Looks good.

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 2015-01-22 20:26:54 +0000
3+++ debian/changelog 2015-01-23 14:57:18 +0000
4@@ -1,4 +1,4 @@
5-maas (1.8.0~alpha1+bzr3480-0ubuntu1) UNRELEASED; urgency=medium
6+maas (1.8.0~alpha1+bzr3489-0ubuntu1) UNRELEASED; urgency=medium
7
8 [ Blake Rouse ]
9 * debian/control: added python-pyscss as a build dependency.
10
11=== modified file 'debian/maas-cluster-controller.preinst'
12--- debian/maas-cluster-controller.preinst 2014-10-10 12:07:14 +0000
13+++ debian/maas-cluster-controller.preinst 2015-01-23 14:57:18 +0000
14@@ -26,6 +26,12 @@
15 fi
16 fi
17
18+ if dpkg --compare-versions "$2" lt 1.8.0~alpha1+bzr3486-0ubuntu1; then
19+ if [ -f /var/log/maas/pserv.log ]; then
20+ rm -rf /var/log/maas/pserv.log
21+ fi
22+ fi
23+
24 fi
25
26 #DEBHELPER#
27
28=== modified file 'debian/patches/01-fix-database-settings.patch'
29--- debian/patches/01-fix-database-settings.patch 2014-12-09 16:51:53 +0000
30+++ debian/patches/01-fix-database-settings.patch 2015-01-23 14:57:18 +0000
31@@ -1,6 +1,10 @@
32---- a/contrib/maas_local_settings.py
33-+++ b/contrib/maas_local_settings.py
34-@@ -78,5 +78,5 @@
35+Index: maas-1.8.0~alpha1+bzr3486.orig/contrib/maas_local_settings.py
36+===================================================================
37+--- maas-1.8.0~alpha1+bzr3486.orig.orig/contrib/maas_local_settings.py 2015-01-23 09:50:03.185579822 -0500
38++++ maas-1.8.0~alpha1+bzr3486.orig/contrib/maas_local_settings.py 2015-01-23 09:50:03.181579822 -0500
39+@@ -14,9 +14,9 @@
40+ 'default': {
41+ # 'postgresql_psycopg2', 'postgresql', 'mysql', 'sqlite3' etc.
42 'ENGINE': 'django.db.backends.postgresql_psycopg2',
43 - 'NAME': '',
44 - 'USER': '',
45@@ -9,3 +13,5 @@
46 + 'USER': 'maas',
47 + 'PASSWORD': 'maas',
48 'HOST': 'localhost',
49+ }
50+ }
51
52=== modified file 'debian/patches/02-pserv-config.patch'
53--- debian/patches/02-pserv-config.patch 2015-01-15 10:07:52 +0000
54+++ debian/patches/02-pserv-config.patch 2015-01-23 14:57:18 +0000
55@@ -4,17 +4,10 @@
56 Author: Andres Rodriguez <andreserl@ubuntu.com>
57
58 === modified file 'etc/maas/pserv.yaml'
59---- a/etc/maas/pserv.yaml 2015-01-14 16:24:44 +0000
60-+++ b/etc/maas/pserv.yaml 2015-01-15 10:05:50 +0000
61-@@ -6,7 +6,7 @@
62- ## running server.
63- #
64- # logfile: "pserv.log"
65--logfile: "/dev/null"
66-+logfile: "/var/log/maas/pserv.log"
67-
68- ## TFTP configuration.
69- #
70+Index: maas-1.8.0~alpha1+bzr3486.orig/etc/maas/pserv.yaml
71+===================================================================
72+--- maas-1.8.0~alpha1+bzr3486.orig.orig/etc/maas/pserv.yaml 2015-01-23 09:49:28.057580893 -0500
73++++ maas-1.8.0~alpha1+bzr3486.orig/etc/maas/pserv.yaml 2015-01-23 09:49:49.137580251 -0500
74 @@ -18,8 +18,6 @@
75 # resource_root: /var/lib/maas/boot-resources/current/
76
77@@ -24,4 +17,3 @@
78 # generator: http://localhost/MAAS/api/1.0/pxeconfig/
79 - generator: http://localhost:5243/api/1.0/pxeconfig/
80
81-

Subscribers

People subscribed via source and target branches

to all changes: