From cd942f5deb2d56eecf98016bcdd5a7a626ab2bdb Mon Sep 17 00:00:00 2001
From: "Bryn M. Reeves" <bmr@redhat.com>
Date: Tue, 29 Oct 2013 14:43:02 +0000
Subject: Move sar data colletion to sar plug-in

Historically the general plug-in collected the content of the
/var/log/sa directory while a separate sar plug-in handles the text
formatted summary files.

This makes little sense and adds to the bloat in the "general"
module; move the collection to the sar plug-in itself.

Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
---
 sos/plugins/general.py | 1 -
 sos/plugins/sar.py     | 1 +
 2 files changed, 1 insertion(+), 1 deletion(-)

diff --git a/sos/plugins/general.py b/sos/plugins/general.py
index fbf39917..d7059e66 100644
--- a/sos/plugins/general.py
+++ b/sos/plugins/general.py
@@ -28,7 +28,6 @@ class General(Plugin):
             "/etc/sos.conf",
             "/etc/sysconfig",
             "/proc/stat",
-            "/var/log/sa",
             "/var/log/pm/suspend.log",
             "/var/log/up2date",
             "/etc/hostid",
diff --git a/sos/plugins/sar.py b/sos/plugins/sar.py
index c09055cb..6c093237 100644
--- a/sos/plugins/sar.py
+++ b/sos/plugins/sar.py
@@ -33,6 +33,7 @@ class Sar(Plugin,):
         return True
 
     def setup(self):
+        self.add_copy_spec("/var/log/sa")
         dirList = os.listdir(self.sa_path)
         # find all the sa file that don't have an existing sar file
         for fname in dirList:
-- 
cgit