Skip to content

Commit

Permalink
Fixes #110 in newsapps/beeswithmachineguns, security groups being ign…
Browse files Browse the repository at this point in the history
…ored when -v is specified. Removing the conditional logic on line 210 because it seems to duplicate the intention of line 178, except it defaults to None instead of looking up the sg for the appropriate subnet. In addition this looked suspicious because line 192 for spot/bid instances does not contain such a conditional
  • Loading branch information
Ryan Rawdon committed Nov 15, 2017
1 parent ca2e47f commit 324149d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion beeswithmachineguns/bees.py
Original file line number Diff line number Diff line change
Expand Up @@ -207,7 +207,7 @@ def up(count, group, zone, image_id, instance_type, username, key_name, subnet,
min_count=count,
max_count=count,
key_name=key_name,
security_group_ids=[groupId] if not subnet else None,
security_group_ids=[groupId],
instance_type=instance_type,
placement=placement,
subnet_id=subnet)
Expand Down

0 comments on commit 324149d

Please sign in to comment.