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

Proposed by Andres Rodriguez
Status: Merged
Approved by: Andres Rodriguez
Approved revision: 551
Merged at revision: 551
Proposed branch: lp:~andreserl/maas/lp1677005
Merge into: lp:~maas-maintainers/maas/packaging
Diff against target: 33 lines (+9/-3)
2 files modified
debian/changelog (+3/-1)
debian/maas-common.logrotate (+6/-2)
To merge this branch: bzr merge lp:~andreserl/maas/lp1677005
Reviewer Review Type Date Requested Status
Mike Pontillo (community) Approve
Review via email: mp+321211@code.launchpad.net

Commit message

Improve log rotation for maas.log

Match rotation of maas.log to be weekly (same as regiond.log and rackd.log), but store logs for 8 weeks.

To post a comment you must log in.
Revision history for this message
Mike Pontillo (mpontillo) wrote :

Looks good; thanks!

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 2017-03-28 15:09:30 +0000
3+++ debian/changelog 2017-03-28 19:36:08 +0000
4@@ -1,6 +1,8 @@
5 maas (2.2.0~beta5+bzr5865-0ubuntu1) UNRELEASED; urgency=medium
6
7- * UNRELEASED
8+ * New upstream release, MAAS 2.2.0 Beta 5.
9+ * debian/maas-common.logrotate: Rotate maas.log weekly, instead of daily.
10+ Store logs for 8 weeks. (LP: #1677005)
11
12 -- Andres Rodriguez <andreserl@ubuntu.com> Tue, 28 Mar 2017 11:08:38 -0400
13
14
15=== modified file 'debian/maas-common.logrotate'
16--- debian/maas-common.logrotate 2015-05-14 10:08:23 +0000
17+++ debian/maas-common.logrotate 2017-03-28 19:36:08 +0000
18@@ -1,9 +1,13 @@
19 /var/log/maas/maas.log
20 {
21- rotate 7
22- daily
23+ rotate 8
24+ weekly
25 missingok
26 notifempty
27 delaycompress
28 compress
29+ # Don't rotate unless the log is at least 10MB.
30+ minsize 10M
31+ # Force rotation if the log grows beyond 50MB.
32+ maxsize 50M
33 }

Subscribers

People subscribed via source and target branches

to all changes: