diff options
| author | Jake Hunsaker <jhunsake@redhat.com> | 2019-08-19 13:49:22 -0400 |
|---|---|---|
| committer | Pavel Moravec <pmoravec@redhat.com> | 2019-08-25 12:45:24 +0200 |
| commit | 3240475c54af3fde9857434c936a64debbdf90c2 (patch) | |
| tree | afa6bf786a804a37e0ca1518c3928251991be5d4 | |
| parent | aee752eb016e70ed165c05224d5722b0db07dfb5 (diff) | |
| download | sos-3240475c54af3fde9857434c936a64debbdf90c2.tar.gz | |
[libvirt] Collect rotated qemu logs and remove size limits
Expands qemu log collection to included rotated logs.
Additionally, removes the 5MB limit for each path collection as sos now
has implicit size limiting in place, and 5MB would be too small of a
limit for systems running a large number of VMs (such as OSP or oVirt
nodes).
Resolves: #1758
Signed-off-by: Jake Hunsaker <jhunsake@redhat.com>
Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
| -rw-r--r-- | sos/plugins/libvirt.py | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/sos/plugins/libvirt.py b/sos/plugins/libvirt.py index 07052af3..00a63f2e 100644 --- a/sos/plugins/libvirt.py +++ b/sos/plugins/libvirt.py @@ -46,10 +46,12 @@ class Libvirt(Plugin, RedHatPlugin, UbuntuPlugin, DebianPlugin): ]) if not self.get_option("all_logs"): - self.add_copy_spec("/var/log/libvirt/libvirtd.log", sizelimit=5) - self.add_copy_spec("/var/log/libvirt/qemu/*.log", sizelimit=5) - self.add_copy_spec("/var/log/libvirt/lxc/*.log", sizelimit=5) - self.add_copy_spec("/var/log/libvirt/uml/*.log", sizelimit=5) + self.add_copy_spec([ + "/var/log/libvirt/libvirtd.log", + "/var/log/libvirt/qemu/*.log*", + "/var/log/libvirt/lxc/*.log", + "/var/log/libvirt/uml/*.log" + ]) else: self.add_copy_spec("/var/log/libvirt") |