roles/common/tasks/main.yml: Trailing `/' on copy destination directories.

Ansible correctly notices this and creates the necessary directory if it
doesn't exist already.  Without the trailing `/', Ansible might copy the
file to where we wanted the directory and then everything is weird.

diff --git a/roles/common/tasks/main.yml b/roles/common/tasks/main.yml
index bfad5558936031844b8a81a5d11a0afada1fa317..9c2f2f1e8631e4c8ea6e21d103ef1b3fb2dc21e2 100644 (file)
--- a/roles/common/tasks/main.yml
+++ b/roles/common/tasks/main.yml
@@ -16,14 +16,14 @@
 
 - name: install PKI maintenance scripts
   tags: [pki, pki-scripts]
 
 - name: install PKI maintenance scripts
   tags: [pki, pki-scripts]

-  copy: src=pki/{{ item }} dest=/etc/cron.daily
+  copy: src=pki/{{ item }} dest=/etc/cron.daily/

   with_items:
     - update-ca-certs
     - check-x509-certs
 
 - name: install common PKI files
   tags: [pki, pki-keys]
   with_items:
     - update-ca-certs
     - check-x509-certs
 
 - name: install common PKI files
   tags: [pki, pki-keys]

-  copy: src=pki/{{ item }} dest=/etc/ca
+  copy: src=pki/{{ item }} dest=/etc/ca/

   with_items:
     - ca.cert
     - dh-param.pem
   with_items:
     - ca.cert
     - dh-param.pem


@@ -39,7 +39,7 @@
 
 - name: install NTP client configuration files
   tags: [ntp, ntp-client]
 
 - name: install NTP client configuration files
   tags: [ntp, ntp-client]

-  copy: src=ntp-client/ntp.conf dest=/etc
+  copy: src=ntp-client/ntp.conf dest=/etc/

   when: ('ntp') not in server |default([])
   notify: restart ntpd
 
   when: ('ntp') not in server |default([])
   notify: restart ntpd
 


@@ -48,7 +48,7 @@
 
 - name: install netdb files
   tags: netdb
 
 - name: install netdb files
   tags: netdb

-  copy: src=netdb/{{ item }} dest=/etc
+  copy: src=netdb/{{ item }} dest=/etc/

   with_items:
     - hosts
     - networks
   with_items:
     - hosts
     - networks


@@ -59,7 +59,7 @@
 
 - name: install SSH configuration files
   tags: [ssh, ssh-config]
 
 - name: install SSH configuration files
   tags: [ssh, ssh-config]

-  copy: src=ssh-config/{{ item }} dest=/etc/ssh
+  copy: src=ssh-config/{{ item }} dest=/etc/ssh/

   notify: restart ssh
   with_items:
     - Makefile
   notify: restart ssh
   with_items:
     - Makefile


@@ -74,7 +74,7 @@
 
 - name: install keys for root SSH access
   tags: [ssh, ssh-root]
 
 - name: install keys for root SSH access
   tags: [ssh, ssh-root]

-  copy: src=ssh-root/{{ item }} dest=/root/.ssh
+  copy: src=ssh-root/{{ item }} dest=/root/.ssh/

   notify: make in /root/.ssh
   with_items:
     - Makefile
   notify: make in /root/.ssh
   with_items:
     - Makefile


@@ -93,7 +93,7 @@
 
 - name: install required backup scripts on non-Debian hosts
   tags: [backup, backup-scripts]
 
 - name: install required backup scripts on non-Debian hosts
   tags: [backup, backup-scripts]

-  copy: src=backup/fshash dest=/usr/local/bin
+  copy: src=backup/fshash dest=/usr/local/bin/

   when: os != 'debian'
 
 ###--------------------------------------------------------------------------
   when: os != 'debian'
 
 ###--------------------------------------------------------------------------


@@ -101,11 +101,11 @@
 
 - name: install sudo configuration
   tags: [sudo]
 
 - name: install sudo configuration
   tags: [sudo]

-  copy: src=sudo/sudoers dest=/etc
+  copy: src=sudo/sudoers dest=/etc/

 
 - name: install common scripts
   tags: [scripts]
 
 - name: install common scripts
   tags: [scripts]

-  copy: src=scripts/{{ item }} dest=/usr/local/bin
+  copy: src=scripts/{{ item }} dest=/usr/local/bin/

   with_items:
     - fetch-unpack-archive
     - genx509
   with_items:
     - fetch-unpack-archive
     - genx509