Merge pull request #157 from rightscale-cookbooks-contrib/ip_missing_79

closes #79, does not include host if ip is null.

CHANGELOG.md

@@ -5,6 +5,7 @@ This file is used to list changes made in each version of the haproxy cookbook.
 ## v3.0.1 (tbc)
 
 - adding tproxy and splice per [#98][]
+- removing members with nil ips from member array. [#79][]
 
 ## v3.0.0 (2017-1-24)
 

recipes/app_lb.rb

@@ -23,7 +23,11 @@
 pool = ["option httpchk #{node['haproxy']['httpchk']}"] if node['haproxy']['httpchk']
 
 servers = node['haproxy']['pool_members'].uniq.map do |s|
-  "#{s[:hostname]} #{s[:ipaddress]}:#{node['haproxy']['member_port']} weight 1 maxconn #{node['haproxy']['member_max_connections']} check"
+  if !s[:ipaddress].nil?
+    "#{s[:hostname]} #{s[:ipaddress]}:#{node['haproxy']['member_port']} weight 1 maxconn #{node['haproxy']['member_max_connections']} check"
+  else
+    Chef::Log.info "haproxy_host_missing_ip: #{s[:hostname]}"
+  end
 end
 
 haproxy_lb (node['haproxy']['mode']).to_s do