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

Proposed by Andres Rodriguez
Status: Merged
Approved by: Andres Rodriguez
Approved revision: 311
Merged at revision: 311
Proposed branch: lp:~andreserl/maas/packaging_fix_proxy_log_permissions
Merge into: lp:~maas-maintainers/maas/packaging
Diff against target: 46 lines (+14/-0)
3 files modified
debian/maas-proxy.postinst (+4/-0)
debian/maas-region-controller-min.postinst (+5/-0)
debian/maas-region-controller.postinst (+5/-0)
To merge this branch: bzr merge lp:~andreserl/maas/packaging_fix_proxy_log_permissions
Reviewer Review Type Date Requested Status
Andres Rodriguez (community) Approve
Review via email: mp+235839@code.launchpad.net

Commit message

Ensure maas proxy log dir has correct permissions

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

self approve

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/maas-proxy.postinst'
2--- debian/maas-proxy.postinst 2014-09-21 22:05:31 +0000
3+++ debian/maas-proxy.postinst 2014-09-24 17:17:48 +0000
4@@ -8,6 +8,10 @@
5 # We run our own squid3 instances.
6 invoke-rc.d squid3 stop
7
8+ # Ensure log folder is created.
9+ mkdir -p /var/log/maas/proxy
10+ chown -R proxy:proxy /var/log/maas/proxy
11+
12 fi
13
14 #DEBHELPER#
15
16=== modified file 'debian/maas-region-controller-min.postinst'
17--- debian/maas-region-controller-min.postinst 2014-09-12 19:10:04 +0000
18+++ debian/maas-region-controller-min.postinst 2014-09-24 17:17:48 +0000
19@@ -120,6 +120,11 @@
20 # Make sure rsyslog reads our config
21 invoke-rc.d rsyslog restart
22
23+ # If proxy log dir exists, set correct permissions
24+ if [ -d /var/log/maas/proxy ]; then
25+ chown -R proxy:proxy /var/log/maas/proxy
26+ fi
27+
28 elif [ -n "$DEBCONF_RECONFIGURE" ]; then
29 # Set the IP address of the interface with default route
30 db_get maas/default-maas-url
31
32=== modified file 'debian/maas-region-controller.postinst'
33--- debian/maas-region-controller.postinst 2014-09-12 19:10:04 +0000
34+++ debian/maas-region-controller.postinst 2014-09-24 17:17:48 +0000
35@@ -150,6 +150,11 @@
36 chown -R syslog:syslog /var/log/maas/rsyslog
37 # Make sure rsyslog reads our config
38 invoke-rc.d rsyslog restart
39+
40+ # If proxy log dir exists, set correct permissions
41+ if [ -d /var/log/maas/proxy ]; then
42+ chown -R proxy:proxy /var/log/maas/proxy
43+ fi
44 }
45
46 configure_third_party_log_symlinks() {

Subscribers

People subscribed via source and target branches

to all changes: