search

nathanmarz / storm-deploy

One click deploy for Storm clusters on AWS
Other
516 stars 148 forks source link

Fix for #39. #40

Closed xpe closed 10 years ago

xpe commented 11 years ago

I think that the upgrade from jClouds 1.4.2 to 1.5.1 in the previous commit (9d6e47b7) made a change in how .createSecurityGroupInRegion works (see security.clj).

lorcan commented 11 years ago

This also fixed the issue for me. Thanks :-) Can we get this request pulled in?

msegel commented 10 years ago

Hi, ran in to the same problem... found the fix in lorcan's branch (which looks like xpe found first)... What does it take to get it implemented in the main branch?

Looks like it was fixed 2 months ago?

Thx

64BitChris commented 10 years ago

Can we please merge this in? As it stands, the master branch is unusable without this fix. I had to merge in @lorcan 's fix to get this to work.

64BitChris commented 10 years ago

Thanks so much!!

xpe commented 10 years ago

@tbatchelli Thank you!