search

nathanmarz / storm-deploy

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

Fixed IP Address Output to properly list all the IPs for the various nodes #23

Closed txbm closed 11 years ago

txbm commented 11 years ago

tl;dr Fix for issue #21.

For some reason the listing code was not joining the items inside of the LazySeq returned by the map function being used to gather the primary and private IPs for the nodes.

Adding in the join call on the resulting LazySeq produces a comma-separated list of IPs for convenient summary during deploy and on-demand with lein run :deploy --ips

This merge also contains some random parenthetical cleanup from a previous commit.

Fully tested and working.

jasonjckn commented 11 years ago

thanks.