Addressing tox issues

roles/openshift_sanitize_inventory/filter_plugins/openshift_sanitize_inventory.py

@@ -1,10 +1,11 @@
-import re
-
 '''
  Openshift Sanitize inventory class that provides useful filters used in Logging.
 '''
 
 
+import re
+
+
 # This should be removed after map_from_pairs is no longer used in __deprecations_logging.yml
 def map_from_pairs(source, delim="="):
     ''' Returns a dict given the source and delim delimited '''
@@ -25,7 +26,7 @@ def vars_with_pattern(source, pattern=""):
 
     for item in source:
         if var_pattern.match(item):
-          var_list.append(item)
+            var_list.append(item)
 
     return var_list
 

roles/openshift_sanitize_inventory/tasks/unsupported.yml

@@ -20,13 +20,15 @@
 
 - name: Ensure that dynamic provisioning is set if using dynamic storage
   when:
+  - dynamic_volumes_check | default(true) | bool
   - not openshift_master_dynamic_provisioning_enabled | default(false) | bool
   - not openshift_cloudprovider_kind is defined
-  - __using_dynamic | bool
+  - __using_dynamic is defined and __using_dynamic | bool
   fail:
     msg: |-
       Using a storage kind of 'dynamic' without enabling dynamic provisioning nor
       setting a cloud provider will cause generated PVCs to not be able to bind as
       intended. Either update to not use a dynamic storage or set
       openshift_master_dynamic_provisioning_enabled to True and set an
-      openshift_cloudprovider_kind.
+      openshift_cloudprovider_kind. You can disable this check with
+      'dynamic_volumes_check=False'.