Renamed NotContainerized to NotContainerizedMixin and dropped no-member

roles/openshift_health_checker/openshift_checks/__init__.py

@@ -50,6 +50,7 @@ class OpenShiftCheck(object):
     @classmethod
     def subclasses(cls):
         """Returns a generator of subclasses of this class and its subclasses."""
+        # AUDIT: no-member makes sense due to this having a metaclass
         for subclass in cls.__subclasses__():  # pylint: disable=no-member
             yield subclass
             for subclass in subclass.subclasses():

roles/openshift_health_checker/openshift_checks/mixins.py

@@ -2,17 +2,14 @@
 from openshift_checks import get_var
 
 
-class NotContainerized(object):
+class NotContainerizedMixin(object):
     """Mixin for checks that are only active when not in containerized mode."""
 
     @classmethod
     def is_active(cls, task_vars):
         return (
-            # This mixin is meant to be used with subclasses of
-            # OpenShiftCheck. Pylint disables this by default on mixins,
-            # though it relies on the class name ending in 'mixin'.
-            # pylint: disable=no-member
-            super(NotContainerized, cls).is_active(task_vars) and
+            # This mixin is meant to be used with subclasses of OpenShiftCheck.
+            super(NotContainerizedMixin, cls).is_active(task_vars) and
             not cls.is_containerized(task_vars)
         )
 

roles/openshift_health_checker/openshift_checks/package_availability.py

@@ -1,9 +1,9 @@
 # pylint: disable=missing-docstring
 from openshift_checks import OpenShiftCheck, get_var
-from openshift_checks.mixins import NotContainerized
+from openshift_checks.mixins import NotContainerizedMixin
 
 
-class PackageAvailability(NotContainerized, OpenShiftCheck):
+class PackageAvailability(NotContainerizedMixin, OpenShiftCheck):
     """Check that required RPM packages are available."""
 
     name = "package_availability"

roles/openshift_health_checker/openshift_checks/package_update.py

@@ -1,9 +1,9 @@
 # pylint: disable=missing-docstring
 from openshift_checks import OpenShiftCheck
-from openshift_checks.mixins import NotContainerized
+from openshift_checks.mixins import NotContainerizedMixin
 
 
-class PackageUpdate(NotContainerized, OpenShiftCheck):
+class PackageUpdate(NotContainerizedMixin, OpenShiftCheck):
     """Check that there are no conflicts in RPM packages."""
 
     name = "package_update"

roles/openshift_health_checker/openshift_checks/package_version.py

@@ -1,9 +1,9 @@
 # pylint: disable=missing-docstring
 from openshift_checks import OpenShiftCheck, get_var
-from openshift_checks.mixins import NotContainerized
+from openshift_checks.mixins import NotContainerizedMixin
 
 
-class PackageVersion(NotContainerized, OpenShiftCheck):
+class PackageVersion(NotContainerizedMixin, OpenShiftCheck):
     """Check that available RPM packages match the required versions."""
 
     name = "package_version"