Merge lp:~louis/maas/lp1346703_rsyslog_ownership into lp:~maas-maintainers/maas/packaging

[Louis Bouchard]

During installation, don't make the ‘maas’ user owner of log files that are owned by syslog; it stops syslog from writing to them.

maas-region-controller.postinst sets the ownership of the /var/log/maas/rsyslog directory to syslog:syslog so the syslog daemon can write to it.

When maas-cluster-controller.postinst runs after installation of maas-region-controller, it changes the ownership of /var/log/maas/rsyslog to maas:maas. The syslog daemon can no longer write to it.

This proposed fix keeps the /var/log/maas/rsyslog ownership intact

[lp:~louis/maas/lp1346703_rsyslog_ownership]

303. By Louis Bouchard on 2014-09-10

During installation, don't make the ‘maas’ user owner of log files that are owned by syslog; it stops syslog from writing to them.

[Gavin Panella (allenap)]

Tip top.

review: Approve

[lp:~louis/maas/lp1346703_rsyslog_ownership]

304. By Louis Bouchard on 2014-09-10

Change cmd syntax according to Merge Proposal comment

[Andres Rodriguez (andreserl)]

The changelong entry should have been done on UNRELEASED rather than a completely new changelog entry.

[Gavin Panella (allenap)]

Sorry Andres, I didn't realise when reviewing. Next time I'll add you as a reviewer before landing.

=== modified file 'debian/changelog'
--- debian/changelog	2014-09-10 07:54:58 +0000
+++ debian/changelog	2014-09-10 14:08:31 +0000
@@ -1,3 +1,12 @@
+maas (1.7.0~beta1+bzr2781-0ubuntu2) utopic; urgency=medium
+
+  [Louis Bouchard]
+  * debian/maas-cluster-controller.postinst:
+    - Exclude /var/log/maas/rsyslog when changing ownership
+      (LP: #1346703)
+
+ -- Louis Bouchard <[e-mail]>  Tue, 09 Sep 2014 15:53:05 +0200
+
 maas (1.7.0~beta1+bzr2783-0ubuntu2) UNRELEASED; urgency=medium
 
   * UNRELEASED

=== modified file 'debian/maas-cluster-controller.postinst'
--- debian/maas-cluster-controller.postinst	2014-07-29 10:51:48 +0000
+++ debian/maas-cluster-controller.postinst	2014-09-10 14:08:31 +0000
@@ -22,7 +22,7 @@
 
     # Give appropriate permissions
     chown -R maas:maas /var/lib/maas/
-    chown -R maas:maas /var/log/maas
+    find /var/log/maas -not -user syslog -print0 | xargs -r0 chown maas:maas
     chmod -R 775 /var/log/maas/oops
     chown syslog:syslog /var/log/maas/maas.log
 }