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

Merged
merged 2 commits into from Feb 6, 2013

Conversation

Projects
None yet
2 participants
Contributor

petermelias commented Feb 1, 2013

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 jasonjckn added a commit that referenced this pull request Feb 6, 2013

@jasonjckn jasonjckn Merge pull request #23 from petermelias/master
Fixed IP Address Output to properly list all the IPs for the various nodes
ca68f08

@jasonjckn jasonjckn merged commit ca68f08 into nathanmarz:master Feb 6, 2013

Collaborator

jasonjckn commented Feb 6, 2013

thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment