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

Merge pull request #698 from joelthompson/devel

Browse source 
ec2_group with purge_rules will purge too many rules.
This commit is contained in:
Toshio Kuratomi 2015-02-21 06:42:20 -08:00
commit c88259077e
1 changed files with 21 additions and 23 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
cloud/amazon/ec2_group.py
View file

@ -128,7 +128,7 @@ def make_rule_key(prefix, rule, group_id, cidr_ip):
def addRulesToLookup(rules, prefix, dict): def addRulesToLookup(rules, prefix, dict):
for rule in rules: for rule in rules:
for grant in rule.grants: for grant in rule.grants:
dict[make_rule_key(prefix, rule, grant.group_id, grant.cidr_ip)] = rule dict[make_rule_key(prefix, rule, grant.group_id, grant.cidr_ip)] = (rule, grant)
def get_target_from_rule(module, ec2, rule, name, group, groups, vpc_id): def get_target_from_rule(module, ec2, rule, name, group, groups, vpc_id):
@ -304,8 +304,7 @@ def main():
# Finally, remove anything left in the groupRules -- these will be defunct rules # Finally, remove anything left in the groupRules -- these will be defunct rules
if purge_rules: if purge_rules:
for rule in groupRules.itervalues() : for (rule, grant) in groupRules.itervalues() :
for grant in rule.grants:
grantGroup = None grantGroup = None
if grant.group_id: if grant.group_id:
grantGroup = groups[grant.group_id] grantGroup = groups[grant.group_id]
@ -369,8 +368,7 @@ def main():
# Finally, remove anything left in the groupRules -- these will be defunct rules # Finally, remove anything left in the groupRules -- these will be defunct rules
if purge_rules_egress: if purge_rules_egress:
for rule in groupRules.itervalues(): for (rule, grant) in groupRules.itervalues():
for grant in rule.grants:
grantGroup = None grantGroup = None
if grant.group_id: if grant.group_id:
grantGroup = groups[grant.group_id].id grantGroup = groups[grant.group_id].id