Bug 526769

Summary: libvirt logrotate settings should roll over weekly instead of daily
Product: [Fedora] Fedora Reporter: Cole Robinson <crobinso>
Component: libvirtAssignee: Mark McLoughlin <markmc>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: high    
Version: rawhideCC: berrange, clalance, crobinso, itamar, jforbes, markmc, veillard, virt-maint
Target Milestone: ---Keywords: Reopened
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2009-10-06 15:23:27 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 498969    

Description Cole Robinson 2009-10-01 17:46:42 UTC
Since we no longer log qemu monitor output to the /var/log/libvirt/qemu logfile, there isn't much chance of these log files becoming huge in short periods of time. Using aggressive logrotate settings then provides little benefit at the cost of debug ability (for example, bug 524508c#12 ).

I think weekly, biweekly, or even monthly rotation should be enough, maybe removing compression altogether.

Comment 1 Daniel Veillard 2009-10-02 11:22:43 UTC
I would really keep compression in any case, as this leads to no loss of
informations.
Now how much information we keep by default is a trade-off, honestly
keeping them for a week should be sufficient to diagnose problem, but
I would be okay to switch that to weekly and rotate 4 to keep a month
worth of data instead of a week.

Daniel

Comment 2 Daniel Veillard 2009-10-02 11:25:54 UTC
this would also limit the number of entries in that directory, cut this
nearly by half for the same amount of VM, currently there is 8 files
per domain, usually of zero lenght if inactive, and we would lower this
to 5 files but bigger.

Daniel

Comment 3 Mark McLoughlin 2009-10-02 15:43:01 UTC
Yeah, this has been annoying me lately too

Let's switch to weekly

Comment 4 Daniel Veillard 2009-10-02 16:09:35 UTC
I posted the trivial patch upstream

Daniel

Comment 5 Daniel Veillard 2009-10-05 15:09:11 UTC
Okay fixed upstream, it will reach rawhide in a couple of weeks I think

Daniel

Comment 6 Mark McLoughlin 2009-10-05 15:21:18 UTC
We should pull this into F-12 before GA

Comment 7 Mark McLoughlin 2009-10-06 09:46:18 UTC
* Tue Oct  6 2009 Mark McLoughlin <markmc> - 0.7.1-9
- Change logrotate config to weekly (#526769)

Comment 8 Mark McLoughlin 2009-10-06 13:03:31 UTC
tag request : https://fedorahosted.org/rel-eng/ticket/2354

Comment 9 Mark McLoughlin 2009-10-06 15:23:27 UTC
tagged now