From a9e85db3512252a3996680572baf8e8422f148f1 Mon Sep 17 00:00:00 2001 From: Steve Milner Date: Thu, 12 Jan 2017 16:15:37 -0500 Subject: filter: Removed unused validation calls The parent class for filters in openshift_master defines a validate method which does nothing. This change removes calls to this method while leaving the validation implementations as is. --- filter_plugins/openshift_master.py | 3 --- 1 file changed, 3 deletions(-) diff --git a/filter_plugins/openshift_master.py b/filter_plugins/openshift_master.py index 6299369b3..437f4c400 100644 --- a/filter_plugins/openshift_master.py +++ b/filter_plugins/openshift_master.py @@ -176,7 +176,6 @@ class LDAPPasswordIdentityProvider(IdentityProviderBase): def validate(self): ''' validate this idp instance ''' - super(self.__class__, self).validate() if not isinstance(self.provider['attributes'], dict): raise errors.AnsibleFilterError("|failed attributes for provider " "{0} must be a dictionary".format(self.__class__.__name__)) @@ -238,7 +237,6 @@ class RequestHeaderIdentityProvider(IdentityProviderBase): def validate(self): ''' validate this idp instance ''' - super(self.__class__, self).validate() if not isinstance(self.provider['headers'], list): raise errors.AnsibleFilterError("|failed headers for provider {0} " "must be a list".format(self.__class__.__name__)) @@ -341,7 +339,6 @@ class IdentityProviderOauthBase(IdentityProviderBase): def validate(self): ''' validate this idp instance ''' - super(self.__class__, self).validate() if self.challenge: raise errors.AnsibleFilterError("|failed provider {0} does not " "allow challenge authentication".format(self.__class__.__name__)) -- cgit v1.2.3