aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBryn M. Reeves <bmr@redhat.com>2014-03-12 19:36:02 +0000
committerBryn M. Reeves <bmr@redhat.com>2014-03-12 19:36:02 +0000
commitdb8839351479c60234bb6873394d93b56f0174eb (patch)
tree5f4c3bcfacdcd331ad41cf4933f5bc84225f3d05
parent15f46d44afa055edc169670303c81fb97dcfd0ae (diff)
downloadsos-db8839351479c60234bb6873394d93b56f0174eb.tar.gz
Rename validatePlugin to validate_plugin
This one somehow didn't get the memo about camelCase not being cool any more in sos. Rename the method to comply with pep8 and not be so ugly. Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
-rw-r--r--sos/policies/__init__.py2
-rw-r--r--sos/sosreport.py2
2 files changed, 2 insertions, 2 deletions
diff --git a/sos/policies/__init__.py b/sos/policies/__init__.py
index c272e5c6..b91d0fd8 100644
--- a/sos/policies/__init__.py
+++ b/sos/policies/__init__.py
@@ -198,7 +198,7 @@ No changes will be made to system configuration.
return tempfile.gettempdir()
return opt_tmp_dir
- def validatePlugin(self, plugin_class):
+ def validate_plugin(self, plugin_class):
"""
Verifies that the plugin_class should execute under this policy
"""
diff --git a/sos/sosreport.py b/sos/sosreport.py
index 0faa3642..fe78abde 100644
--- a/sos/sosreport.py
+++ b/sos/sosreport.py
@@ -763,7 +763,7 @@ class SoSReport(object):
tuple(self.policy.valid_subclasses))
for plugin_class in plugin_classes:
- if not self.policy.validatePlugin(plugin_class):
+ if not self.policy.validate_plugin(plugin_class):
self.soslog.warning(_("plugin %s does not validate, skipping") % plug)
if self.opts.verbosity > 0:
self._skip(plugin_class, _("does not validate"))