Adds the check for 'not None' also when building host_info dict for ElastiCache clusters, nodes and replication groups

This commit is contained in:
Victor Schröder 2015-06-29 23:28:55 +02:00
parent d88a42570e
commit df77d087a5

View file

@ -1117,10 +1117,14 @@ class Ec2Inventory(object):
# Target: Almost everything
elif key == 'ec2_security_groups':
sg_ids = []
for sg in value:
sg_ids.append(sg['SecurityGroupId'])
host_info["ec2_security_group_ids"] = ','.join([str(i) for i in sg_ids])
# Skip if SecurityGroups is None
# (it is possible to have the key defined but no value in it).
if value is not None:
sg_ids = []
for sg in value:
sg_ids.append(sg['SecurityGroupId'])
host_info["ec2_security_group_ids"] = ','.join([str(i) for i in sg_ids])
# Target: Everything
# Preserve booleans and integers