Fixes baseclass for new f5 modules (#22333)
This commit is contained in:
parent
d942196666
commit
6ff9bd0a33
1 changed files with 12 additions and 41 deletions
|
@ -282,17 +282,12 @@ class AnsibleF5Client(object):
|
||||||
)
|
)
|
||||||
|
|
||||||
|
|
||||||
class AnsibleF5Parameters(with_metaclass(ABCMeta, object)):
|
class AnsibleF5Parameters(object):
|
||||||
def __init__(self, params=None):
|
def __init__(self, params=None):
|
||||||
self._values = defaultdict(lambda: None)
|
self._values = defaultdict(lambda: None)
|
||||||
if params:
|
if params:
|
||||||
for k in params:
|
for k,v in iteritems(params):
|
||||||
self._values[k] = params[k]
|
setattr(self, k, v)
|
||||||
|
|
||||||
@abstractproperty
|
|
||||||
def param_api_map(self):
|
|
||||||
"""Dict used to map module parameters to API parameters"""
|
|
||||||
pass
|
|
||||||
|
|
||||||
@property
|
@property
|
||||||
def partition(self):
|
def partition(self):
|
||||||
|
@ -300,47 +295,23 @@ class AnsibleF5Parameters(with_metaclass(ABCMeta, object)):
|
||||||
return 'Common'
|
return 'Common'
|
||||||
return self._values['partition'].strip('/')
|
return self._values['partition'].strip('/')
|
||||||
|
|
||||||
@partition.setter
|
|
||||||
def partition(self, value):
|
|
||||||
self._values['partition'] = value
|
|
||||||
|
|
||||||
@classmethod
|
|
||||||
def from_api(cls, params):
|
|
||||||
"""Create Parameters instance from values return by the API
|
|
||||||
|
|
||||||
The API returns values found on the "values" side of the
|
|
||||||
param_api_map dictionary. These need to be mapped to the names
|
|
||||||
of keys expected by the Parameters class (the "key" side of
|
|
||||||
the param_api_map dictionary)
|
|
||||||
|
|
||||||
:param params:
|
|
||||||
:return:
|
|
||||||
"""
|
|
||||||
for key,value in iteritems(cls.param_api_map):
|
|
||||||
params[key] = params.pop(value, None)
|
|
||||||
p = cls(params)
|
|
||||||
return p
|
|
||||||
|
|
||||||
def __getattr__(self, item):
|
def __getattr__(self, item):
|
||||||
return self._values[item]
|
return self._values[item]
|
||||||
|
|
||||||
def api_params(self):
|
def __setattr__(self, key, value):
|
||||||
return self._filter_params(self._api_params_from_map())
|
if key == '_values':
|
||||||
|
self.__dict__['_values'] = value
|
||||||
|
else:
|
||||||
|
super(AnsibleF5Parameters, self).__setattr__(key, value)
|
||||||
|
|
||||||
def _api_params_from_map(self):
|
@partition.setter
|
||||||
result = dict()
|
def partition(self, value):
|
||||||
for k,v in iteritems(self.param_api_map):
|
self._values['partition'] = value
|
||||||
value = getattr(self, k)
|
|
||||||
result[v] = value
|
|
||||||
return result
|
|
||||||
|
|
||||||
def to_return(self):
|
|
||||||
result = self._values
|
|
||||||
return self._filter_params(result)
|
|
||||||
|
|
||||||
def _filter_params(self, params):
|
def _filter_params(self, params):
|
||||||
return dict((k, v) for k, v in iteritems(params) if v is not None)
|
return dict((k, v) for k, v in iteritems(params) if v is not None)
|
||||||
|
|
||||||
|
|
||||||
|
|
||||||
class F5ModuleError(Exception):
|
class F5ModuleError(Exception):
|
||||||
pass
|
pass
|
||||||
|
|
Loading…
Reference in a new issue