rackspace-cookbooks / redis-multi

Chef wrapper cookbook to create redis master/slave server setups
Apache License 2.0
6 stars 6 forks source link

Give redis-multi the ability to use cloud networks #6

Open boxidau opened 9 years ago

boxidau commented 9 years ago

best_ip_for() returns service net IP addresses

https://github.com/jarosser06/chef-sugar-rackspace

Gives the ability to use private networks

martinb3 commented 9 years ago

I'm concerned that community / non-Rackspace users would end up hitting some of the fail calls in chef-sugar-rackspace, which fail if cloud networks are not present. How about the ability to pass a block to redis-multi to use any code to get the IP of another node? Then the caller could simply pass best_rackspace_ip_for as the block.

boxidau commented 9 years ago

Perhaps we could just do cloud_networks() ourselves and if it's result is empty use best_ip_for instead of best_rackspace_ip_for