Don't do invasive Fuse or BRMS scans by default

Remove Fuse and BRMS from default fact list. Also remove JBOSS_FACTS, since it became redundant with JBOSS_FACTS_TUPLE.
quipucords · Oct 17, 2017 · 1d287a2 · 1d287a2
1 parent ed90ebb
commit 1d287a2
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/rho/scancommand.py b/rho/scancommand.py
@@ -464,7 +464,7 @@ def _validate_options(self):
         elif facts == ['all']:
             self.facts_to_collect = list(utilities.ALL_FACTS)
         elif facts == ['jboss']:
-            self.facts_to_collect = list(utilities.JBOSS_FACTS +
+            self.facts_to_collect = list(utilities.JBOSS_FACTS_TUPLE +
                                          utilities.CONNECTION_FACTS_TUPLE)
         elif facts == ['rhel']:
             self.facts_to_collect = list(utilities.RHEL_FACTS +

diff --git a/rho/utilities.py b/rho/utilities.py
@@ -136,11 +136,10 @@
     + CPU_FACTS_TUPLE + SYSID_FACTS_TUPLE + INSTNUM_FACTS_TUPLE \
     + REDHAT_RELEASE_FACTS_TUPLE + UNAME_FACTS_TUPLE
 
-JBOSS_FACTS = JBOSS_FACTS_TUPLE + BRMS_FACTS_TUPLE + FUSE_FACTS_TUPLE
+DEFAULT_FACTS = RHEL_FACTS + JBOSS_FACTS_TUPLE + CONNECTION_FACTS_TUPLE
 
-DEFAULT_FACTS = RHEL_FACTS + JBOSS_FACTS + CONNECTION_FACTS_TUPLE
-
-ALL_FACTS = DEFAULT_FACTS + JBOSS_SCAN_FACTS_TUPLE
+ALL_FACTS = DEFAULT_FACTS + JBOSS_SCAN_FACTS_TUPLE + BRMS_FACTS_TUPLE \
+    + FUSE_FACTS_TUPLE
 
 # 'log' is a convenience for getting the appropriate logger from the
 # logging module. Use it like this: