cmueller/ansible
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

[cloud] make ec2 paging() more functional and add paging to ec2_lc_facts (#27355)

Browse source
This commit is contained in:
Sloane Hertel 2017-08-01 15:06:46 -04:00 committed by Ryan Brown
parent ad2c1e1838
commit d8eccdda3a
2 changed files with 11 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
lib/ansible/module_utils/ec2.py
View file

@ -312,7 +312,7 @@ def ec2_connect(module):
return ec2
def paging(pause=0, marker_property='marker'):
def paging(pause=0, marker_property='marker', result_key=None):
""" Adds paging to boto retrieval functions that support a 'marker'
this is configurable as not all boto functions seem to use the
same name.
@ -323,8 +323,12 @@ def paging(pause=0, marker_property='marker'):
marker = None
while True:
try:
new = f(*args, marker=marker, **kwargs)
marker = getattr(new, marker_property)
if marker:
kwargs[marker_property] = marker
new = f(*args, **kwargs)
marker = new.get(marker_property)
if result_key:
new = new[result_key]
results.extend(new)
if not marker:
break

5
lib/ansible/modules/cloud/amazon/ec2_lc_facts.py
View file

@ -170,6 +170,8 @@ try:
except ImportError:
HAS_BOTO3 = False
from ansible.module_utils.ec2 import paging
def list_launch_configs(connection, module):
@ -180,7 +182,8 @@ def list_launch_configs(connection, module):
sort_end = module.params.get('sort_end')
try:
launch_configs = connection.describe_launch_configurations(LaunchConfigurationNames=launch_config_name)
launch_configs = {'LaunchConfigurations': paging(pause=0, marker_property='NextToken', result_key='LaunchConfigurations')
(connection.describe_launch_configurations)(LaunchConfigurationNames=launch_config_name)}
except ClientError as e:
module.fail_json(msg=e.message)