Merge ~cjwatson/launchpad-mojo-specs:lp-ppa-lb-remove-old-haproxy-logs into launchpad-mojo-specs:master

Proposed by Colin Watson
Status: Merged
Approved by: Benjamin Allot
Approved revision: 76473dedc5296a6b0cf33b4cad5f87be51e14543
Merged at revision: 76473dedc5296a6b0cf33b4cad5f87be51e14543
Proposed branch: ~cjwatson/launchpad-mojo-specs:lp-ppa-lb-remove-old-haproxy-logs
Merge into: launchpad-mojo-specs:master
Diff against target: 25 lines (+14/-0)
1 file modified
lp/bundle.yaml (+14/-0)
Reviewer Review Type Date Requested Status
Benjamin Allot Approve
Canonical Launchpad Engineering Pending
Review via email: mp+455695@code.launchpad.net

Commit message

lp: Remove old haproxy logs on ppa-lb

Description of the change

ppa-lb's haproxy logs are very large, and they mostly duplicate those on ppa-apache2. We also don't have disk space to keep them right now, and resizing requires awkward downtime. Just remove logs rather than rotating them, at least for now.

To post a comment you must log in.
Revision history for this message
Benjamin Allot (ballot) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/lp/bundle.yaml b/lp/bundle.yaml
2index 985979f..3e18874 100644
3--- a/lp/bundle.yaml
4+++ b/lp/bundle.yaml
5@@ -1205,6 +1205,20 @@ applications:
6 {%- endif %}
7 options:
8 enable_monitoring: true
9+ # ppa-lb's logs mostly duplicate those of ppa-apache2, and are very
10+ # large. This is the default with "rotate 52" changed to "rotate 0".
11+ logrotate_config: |
12+ /var/log/haproxy.log {
13+ daily
14+ rotate 0
15+ missingok
16+ notifempty
17+ compress
18+ delaycompress
19+ postrotate
20+ /usr/lib/rsyslog/rsyslog-rotate
21+ endscript
22+ }
23 {#- This should be overridden in secrets #}
24 monitoring_password: "haproxy"
25 monitoring_stats_refresh: 3600

Subscribers

People subscribed via source and target branches