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

Improvement for #186 in beeswithmachineguns, accept sg ID and name #200

Closed FliesLikeABrick closed 6 years ago

FliesLikeABrick commented 6 years ago

Improvement for #186 in beeswithmachineguns, to try and help with possible user experience issues. BWMG requires an id for subnets and AMIs, , but a name for security groups. This commit adds a fallback for security group lookup to use the provided name as an id. This could be optimized by checking if the name provided starts with sg- then using it as an ID, but hardcoding this arbitrary string check was avoided.