launchpad-reviewers team mailing list archive

[Colin Watson <cjwatson@xxxxxxxxxxxxx>]

Colin Watson has proposed merging ~cjwatson/lp-signing:rsync-gunicorn-logs into lp-signing:master.

Commit message:
Publish gunicorn logs over rsync

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~cjwatson/lp-signing/+git/lp-signing/+merge/384727

layer-ols-wsgi hardcodes gunicorn logs to /var/log/gunicorn.log, so they aren't under the existing rsync module that publishes /srv/lp-signing/logs/.  Add a special module that publishes only /var/log/gunicorn.log* and not other files in that directory.
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~cjwatson/lp-signing:rsync-gunicorn-logs into lp-signing:master.

diff --git a/charm/lp-signing/templates/lp-signing-rsync.j2 b/charm/lp-signing/templates/lp-signing-rsync.j2
index 97d85fe..cfb0b3a 100644
--- a/charm/lp-signing/templates/lp-signing-rsync.j2
+++ b/charm/lp-signing/templates/lp-signing-rsync.j2
@@ -6,3 +6,11 @@
   read only = true
   hosts allow = {{ log_hosts_allow }}
 
+[lp-signing-gunicorn-logs]
+  path = /var/log
+  filter = include /gunicorn.log* exclude /*
+  comment = LP Signing Gunicorn Logs
+  list = false
+  read only = true
+  hosts allow = {{ log_hosts_allow }}
+