launchpad-reviewers team mailing list archive

[Colin Watson <mp+450787@xxxxxxxxxxxxxxxxxx>]

Colin Watson has proposed merging ~cjwatson/launchpad:charm-ftpmaster-log-retention into launchpad:master.

Commit message:
charm: Retain ftpmaster logs for longer

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~cjwatson/launchpad/+git/launchpad/+merge/450787

This matches current production configuration, and it can be quite useful to be able to dig through older logs when debugging problems affecting the Ubuntu publisher.
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~cjwatson/launchpad:charm-ftpmaster-log-retention into launchpad:master.

diff --git a/charm/launchpad-ftpmaster-publisher/templates/logrotate.conf.j2 b/charm/launchpad-ftpmaster-publisher/templates/logrotate.conf.j2
index 51d1421..b1f1496 100644
--- a/charm/launchpad-ftpmaster-publisher/templates/logrotate.conf.j2
+++ b/charm/launchpad-ftpmaster-publisher/templates/logrotate.conf.j2
@@ -1,6 +1,6 @@
 {{ logs_dir }}/*.log
 {
-    rotate 21
+    rotate 180
     daily
     dateext
     delaycompress
diff --git a/charm/launchpad-ftpmaster-uploader/templates/logrotate.conf.j2 b/charm/launchpad-ftpmaster-uploader/templates/logrotate.conf.j2
index 51d1421..5056d24 100644
--- a/charm/launchpad-ftpmaster-uploader/templates/logrotate.conf.j2
+++ b/charm/launchpad-ftpmaster-uploader/templates/logrotate.conf.j2
@@ -1,6 +1,6 @@
 {{ logs_dir }}/*.log
 {
-    rotate 21
+    rotate 90
     daily
     dateext
     delaycompress