hosts_deny[name] = details[:hosts_deny] || []
- if details[:nodes_deny]
- hosts_deny[name] |= search(:node, details[:nodes_deny]).collect do |n|
- n.ipaddresses(:role => :external)
- end.flatten
- end
+ next unless details[:nodes_deny]
+
+ hosts_deny[name] |= search(:node, details[:nodes_deny]).collect do |n|
+ n.ipaddresses(:role => :external)
+ end.flatten
end
package "rsync"
service "rsync" do
- action [ :enable, :start ]
+ action [:enable, :start]
supports :status => true, :restart => true
end
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