2013/12/13 Kevin Fenzi <kevin@scrye.com>
Some minor nits:

EXCLUDE_LIST='/etc/rsyncd/backup.exclude'

Doesn't seem to be defined anywhere? Should that be in the users
directory and defined?

Theoretically that entry will be read by rsync when the connection on the relevant machine will be accepted and initiated. (each machine has a custom version of that file which is generated by puppet through each node file)
  
'recurse=yes' when adding the directories doesn't seem needed?

$item should be '{{ item }}' :)
(but I can clean that up later when I fix the rest of the playbook)

Both fixed. :-)


--
Cheers,

Andrea

Debian Developer,
Fedora / EPEL packager,
GNOME Sysadmin,
GNOME Foundation Membership & Elections Committee Chairman

Homepage: http://www.gnome.org/~av