Merge lp://qastaging/~afrantzis/lxc-android-config/fix-1633108-logrotate into lp://qastaging/lxc-android-config

Proposed by Alexandros Frantzis
Status: Needs review
Proposed branch: lp://qastaging/~afrantzis/lxc-android-config/fix-1633108-logrotate
Merge into: lp://qastaging/lxc-android-config
Diff against target: 28 lines (+6/-6)
1 file modified
etc/logrotate.d/touch-syslog (+6/-6)
To merge this branch: bzr merge lp://qastaging/~afrantzis/lxc-android-config/fix-1633108-logrotate
Reviewer Review Type Date Requested Status
Brandon Schaefer (community) Approve
Review via email: mp+308393@code.qastaging.launchpad.net

Commit message

Logrotate syslog and auth.log once to avoid losing useful information

Description of the change

Logrotate syslog and auth.log once to avoid losing useful information

I have halved the size limit of each log to maintain the same total maximum size the logs can occupy at any given time. Perhaps we should revisit the size contraints, though.

To post a comment you must log in.
Revision history for this message
Brandon Schaefer (brandontschaefer) wrote :

Possibly we should change the comment about not keeping files? Otherwise lgtm

review: Approve
31. By Alexandros Frantzis

Update logrotate rule comment

Revision history for this message
Alexandros Frantzis (afrantzis) wrote :

> Possibly we should change the comment about not keeping files?

Good point, fixed. Thanks!

Unmerged revisions

31. By Alexandros Frantzis

Update logrotate rule comment

30. By Alexandros Frantzis

Logrotate syslog and auth.log once to avoid losing useful information

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches