newsapps / beeswithmachineguns

A utility for arming (creating) many bees (micro EC2 instances) to attack (load test) targets (web applications).
http://apps.chicagotribune.com/
MIT License
6.43k stars 631 forks source link

Fixes #110 in newsapps/beeswithmachineguns, security groups being ign… #190

Closed FliesLikeABrick closed 6 years ago

FliesLikeABrick commented 6 years ago

Fixes #110 in newsapps/beeswithmachineguns, security groups being ignored 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