ansible-collections/community.general
1
0
Fork 0
mirror of https://github.com/ansible-collections/community.general.git synced 2025-08-22 22:11:44 -07:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Add boto support checks for new ec2_elb_lb attribute fields

Browse source 
Also minor fixes like adjusting version_added fields, etc.
This commit is contained in:
James Cammarata 2014-08-26 10:57:05 -05:00
commit 38eb5453b4
1 changed files with 27 additions and 13 deletions
Download patch file Download diff file Expand all files Collapse all files

40
library/cloud/ec2_elb_lb
View file

@ -98,7 +98,7 @@ options:
- Wait a specified timeout allowing connections to drain before terminating an instance - Wait a specified timeout allowing connections to drain before terminating an instance
required: false required: false
aliases: [] aliases: []
version_added: "1.7" version_added: "1.8"
cross_az_load_balancing: cross_az_load_balancing:
description: description:
- Distribute load across all configured Availablity Zones - Distribute load across all configured Availablity Zones
@ -106,7 +106,7 @@ options:
default: "no" default: "no"
choices: ["yes", "no"] choices: ["yes", "no"]
aliases: [] aliases: []
version_added: "1.7" version_added: "1.8"
extends_documentation_fragment: aws extends_documentation_fragment: aws
""" """
@ -295,8 +295,13 @@ class ElbManager(object):
self._set_elb_listeners() self._set_elb_listeners()
self._set_subnets() self._set_subnets()
self._set_health_check() self._set_health_check()
self._set_connection_draining_timeout() # boto has introduced support for some ELB attributes in
self._set_cross_az_load_balancing() # different versions, so we check first before trying to
# set them to avoid errors
if self._check_attribute_support('connection_draining'):
self._set_connection_draining_timeout()
if self._check_attribute_support('cross_zone_load_balancing'):
self._set_cross_az_load_balancing()
def ensure_gone(self): def ensure_gone(self):
"""Destroy the ELB""" """Destroy the ELB"""
@ -346,16 +351,15 @@ class ElbManager(object):
else: else:
info['listeners'] = [] info['listeners'] = []
info['connection_draining_timeout'] = self.elb_conn.get_lb_attribute(self.name, if self._check_attribute_support('connection_draining'):
'ConnectionDraining').timeout info['connection_draining_timeout'] = self.elb_conn.get_lb_attribute(self.name, 'ConnectionDraining').timeout
is_cross_az_lb_enabled = self.elb_conn.get_lb_attribute(self.name, if self._check_attribute_support('cross_zone_load_balancing'):
'CrossZoneLoadBalancing') is_cross_az_lb_enabled = self.elb_conn.get_lb_attribute(self.name, 'CrossZoneLoadBalancing')
if is_cross_az_lb_enabled:
if is_cross_az_lb_enabled: info['cross_az_load_balancing'] = 'yes'
info['cross_az_load_balancing'] = 'yes' else:
else: info['cross_az_load_balancing'] = 'no'
info['cross_az_load_balancing'] = 'no'
return info return info
@ -582,6 +586,9 @@ class ElbManager(object):
self.elb.configure_health_check(self.elb.health_check) self.elb.configure_health_check(self.elb.health_check)
self.changed = True self.changed = True
def _check_attribute_support(self, attr):
return hasattr(boto.ec2.elb.attributes.LbAttributes(), attr)
def _set_cross_az_load_balancing(self): def _set_cross_az_load_balancing(self):
attributes = self.elb.get_attributes() attributes = self.elb.get_attributes()
if self.cross_az_load_balancing == 'yes': if self.cross_az_load_balancing == 'yes':
@ -665,6 +672,13 @@ def main():
connection_draining_timeout, cross_az_load_balancing, connection_draining_timeout, cross_az_load_balancing,
region=region, **aws_connect_params) region=region, **aws_connect_params)
# check for unsupported attributes for this version of boto
if cross_az_load_balancing and not elb_man._check_attribute_support('cross_zone_load_balancing'):
module.fail_json(msg="You must install boto >= 2.18.0 to use the cross_az_load_balancing attribute")
if connection_draining_timeout and not elb_man._check_attribute_support('connection_draining'):
module.fail_json(msg="You must install boto >= 2.28.0 to use the connection_draining_timeout attribute")
if state == 'present': if state == 'present':
elb_man.ensure_ok() elb_man.ensure_ok()
elif state == 'absent': elif state == 'absent':