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

[Andres Rodriguez]

debian/maas-common.{postinst,logrotate}: Create maas.log if non-existent
and add log rotation, provided maas.log exists both in the Region
and in the Cluster (LP: #1381603)

[Gavin Panella (allenap)]

Looks good, but with a few comments.

review: Approve

[Andres Rodriguez (andreserl)]

review: Approve

Unmerged revisions

=== modified file 'debian/changelog'
--- debian/changelog	2015-05-09 14:03:35 +0000
+++ debian/changelog	2015-05-14 11:03:04 +0000
@@ -1,6 +1,8 @@
-maas (1.8.0~beta6+bzr3867-0ubuntu1) UNRELEASED; urgency=medium
+maas (1.8.0~beta6+bzr3891-0ubuntu1) UNRELEASED; urgency=medium
 
-  * UNRELEASED
+  * debian/maas-common.{postinst,logrotate}: Create maas.log if non-existent
+    and add log rotation, provided maas.log exists both in the Region
+    and in the Cluster (LP: #1381603)
 
  -- Andres Rodriguez <[e-mail]>  Sat, 09 May 2015 15:03:17 +0100
 

=== added file 'debian/maas-common.logrotate'
--- debian/maas-common.logrotate	1970-01-01 00:00:00 +0000
+++ debian/maas-common.logrotate	2015-05-14 11:03:04 +0000
@@ -0,0 +1,9 @@
+/var/log/maas/maas.log
+{
+        rotate 7
+        daily
+        missingok
+        notifempty
+        delaycompress
+        compress
+}

=== modified file 'debian/maas-common.postinst'
--- debian/maas-common.postinst	2014-12-04 19:32:00 +0000
+++ debian/maas-common.postinst	2015-05-14 11:03:04 +0000
@@ -2,6 +2,19 @@
 
 set -e
 
+create_log_dir(){
+    # Main syslog file.
+    if [ ! -f /var/log/maas/maas.log ]; then
+        mkdir -p /var/log/maas
+        touch /var/log/maas/maas.log
+    fi
+
+    # Give appropriate permissions
+    chown -R maas:maas /var/lib/maas/
+    chown syslog:syslog /var/log/maas/maas.log
+
+}
+
 add_user_group(){
 	local user="maas"
 	local group="maas"
@@ -13,5 +26,6 @@
 ln -sf /usr/share/maas/maas-rsyslog.conf /etc/rsyslog.d/99-maas.conf
 
 add_user_group
+create_log_dir
 
 #DEBHELPER#