Merge ~cmisare/launchpad-mojo-specs:master into launchpad-mojo-specs:master

Proposed by Colin Misare
Status: Needs review
Proposed branch: ~cmisare/launchpad-mojo-specs:master
Merge into: launchpad-mojo-specs:master
Diff against target: 20 lines (+3/-2)
1 file modified
lp/bundle.yaml (+3/-2)
Reviewer Review Type Date Requested Status
Ines Almeida Approve
Review via email: mp+455939@code.launchpad.net

Commit message

do not keep rotated haproxy logs on ppa-lb due to limited available space

To post a comment you must log in.
Revision history for this message
Ines Almeida (ines-almeida) wrote :

Thanks for proposing this. I think we will survive with less logs here

There is a comment in the lines above that needs updating. It currently says `and "rotate 52" changed to "rotate 24".` but now that is being changed to `rotate 0` here.

I can propose that in another MP if you don't want to update it in this one

review: Approve
~cmisare/launchpad-mojo-specs:master updated
b37e545... by Colin Misare

update related comment about rotate value as well

Unmerged commits

b37e545... by Colin Misare

update related comment about rotate value as well

Succeeded
[SUCCEEDED] lint:0 (build)
11 of 1 result
a119651... by Colin Misare

do not keep rotated haproxy logs on ppa-lb due to limited available space

Succeeded
[SUCCEEDED] lint:0 (build)
11 of 1 result

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 740e3ff..2e23670 100644
3--- a/lp/bundle.yaml
4+++ b/lp/bundle.yaml
5@@ -1239,12 +1239,13 @@ applications:
6 enable_monitoring: true
7 # ppa-lb's logs mostly duplicate those of ppa-apache2, and are very
8 # large. This is the default with "daily" changed to "hourly", the
9- # addition of "dateext", and "rotate 52" changed to "rotate 24".
10+ # addition of "dateext", and "rotate 52" changed to "rotate 0".
11 logrotate_config: |
12 /var/log/haproxy.log {
13 hourly
14 dateext
15- rotate 24
16+ # save no rotated files due to limited available space
17+ rotate 0
18 missingok
19 notifempty
20 compress

Subscribers

People subscribed via source and target branches