From da1efee21f7a2637efd6214d19695ede1f7fb269 Mon Sep 17 00:00:00 2001 From: Patryk Zawadzki Date: Wed, 26 Feb 2014 13:39:03 +0100 Subject: [PATCH] Properly match existing security group If we don't care about `vpc_id` then neither should the code. --- library/cloud/ec2_group | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/library/cloud/ec2_group b/library/cloud/ec2_group index 34d9b16124..bbbb0fc24e 100644 --- a/library/cloud/ec2_group +++ b/library/cloud/ec2_group @@ -146,7 +146,7 @@ def main(): groups[curGroup.id] = curGroup groups[curGroup.name] = curGroup - if curGroup.name == name and curGroup.vpc_id == vpc_id: + if curGroup.name == name and (vpc_id is None or curGroup.vpc_id == vpc_id): group = curGroup # Ensure requested group is absent