node[:rsyncd][:modules].each do |name, details|
hosts_allow[name] = details[:hosts_allow] || []
- hosts_allow[name] |= search(:node, details[:nodes_allow]).collect do |n|
- n.ipaddresses(:role => :external)
- end.flatten if details[:nodes_allow]
+ if details[:nodes_allow]
+ hosts_allow[name] |= search(:node, details[:nodes_allow]).collect do |n|
+ n.ipaddresses(:role => :external)
+ end.flatten
+ end
hosts_deny[name] = details[:hosts_deny] || []
+ next unless details[:nodes_deny]
+
hosts_deny[name] |= search(:node, details[:nodes_deny]).collect do |n|
n.ipaddresses(:role => :external)
- end.flatten if details[:nodes_deny]
+ end.flatten
end
package "rsync"
source "rsync.erb"
owner "root"
group "root"
- mode 0644
+ mode 0o644
notifies :restart, "service[rsync]"
end
source "rsyncd.conf.erb"
owner "root"
group "root"
- mode 0644
+ mode 0o644
variables :hosts_allow => hosts_allow, :hosts_deny => hosts_deny
end