ansible-collections/community.general
1
0
Fork 0
mirror of https://github.com/ansible-collections/community.general.git synced 2025-06-27 18:50:21 -07:00
Code Issues Projects Releases Packages Wiki Activity Actions 8

adds the ability to set pool member monitors to none (#54549)

Browse source
This commit is contained in:
Wojciech Wypior 2019-03-29 02:06:26 +01:00 • committed by Tim Rupp
parent b6d30a5825
commit 08c7dd94b2
1 changed files with 9 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

11
lib/ansible/modules/network/f5/bigip_pool_member.py
View file

@ -154,6 +154,8 @@ options:
- Specifies, if you activate more than one health monitor, the number of health - Specifies, if you activate more than one health monitor, the number of health
monitors that must receive successful responses in order for the link to be monitors that must receive successful responses in order for the link to be
considered available. considered available.
- Specifying an empty string will remove the monitors and revert to inheriting from pool (default).
- Specifying C(none) value will remove any health monitoring from the member completely.
suboptions: suboptions:
type: type:
description: description:
@ -601,8 +603,10 @@ class ModuleParameters(Parameters):
def monitors(self): def monitors(self):
if self._values['monitors'] is None: if self._values['monitors'] is None:
return None return None
if len(self._values['monitors']) == 1 and self._values['monitors'][0] in ['', 'none']: if len(self._values['monitors']) == 1 and self._values['monitors'][0] == '':
return 'default' return 'default'
if len(self._values['monitors']) == 1 and self._values['monitors'][0] == 'none':
return '/Common/none'
monitors = [fq_name(self.partition, x) for x in self.monitors_list] monitors = [fq_name(self.partition, x) for x in self.monitors_list]
if self.availability_requirement_type == 'at_least': if self.availability_requirement_type == 'at_least':
if self.at_least > len(self.monitors_list): if self.at_least > len(self.monitors_list):
@ -968,6 +972,9 @@ class Difference(object):
return None return None
if self.want.monitors == 'default' and len(self.have.monitors) > 0: if self.want.monitors == 'default' and len(self.have.monitors) > 0:
return 'default' return 'default'
# this is necessary as in v12 there is a bug where returned value has a space at the end
if self.want.monitors == '/Common/none' and self.have.monitors in ['/Common/none', '/Common/none ']:
return None
if self.have.monitors is None: if self.have.monitors is None:
return self.want.monitors return self.want.monitors
if self.have.monitors != self.want.monitors: if self.have.monitors != self.want.monitors:
@ -1147,7 +1154,7 @@ class ModuleManager(object):
result = dict() result = dict()
state = params['state'] state = params['state']
if state in ['present', 'present', 'enabled', 'disabled', 'forced_offline']: if state in ['present', 'enabled', 'disabled', 'forced_offline']:
changed = self.present() changed = self.present()
elif state == "absent": elif state == "absent":
changed = self.absent() changed = self.absent()