More with_fileglob fixes.

playbooks/hosts/copr-be-dev.cloud.fedoraproject.org.yml

@@ -56,7 +56,8 @@
 
   - name: add copr-buildsys keys to copr user path
     action: copy src={{ item }} dest=/home/copr/cloud/ owner=copr group=copr mode=0600
-    with_fileglob: "{{ private }}/files/openstack/copr-copr/*"
+    with_fileglob:
+     - "{{ private }}/files/openstack/copr-copr/*"
 
   - name: setup privkey for copr user
     action: copy src="{{ private }}/files/copr/buildsys.priv" dest=/home/copr/.ssh/id_rsa owner=copr group=copr mode=600
@@ -123,14 +124,16 @@
 
   - name: put files into the files subdir off of provisioning
     action: copy src={{ item }} dest=/home/copr/provision/files/
-    with_fileglob: "{{ files }}/copr/provision/files/*"
+    with_fileglob:
+     - "{{ files }}/copr/provision/files/*"
     tags:
     - provision_config
 
   # ansible lacks a recurse - so we need this until then
   - name: put files into the files/mock subdir off of provisioning
     action: copy src={{ item }} dest=/home/copr/provision/files/mock
-    with_fileglob: "{{ files }}/copr/provision/files/mock/*"
+    with_fileglob:
+     - "{{ files }}/copr/provision/files/mock/*"
     tags:
     - provision_config
 

playbooks/hosts/copr-be.cloud.fedoraproject.org.yml

@@ -73,7 +73,8 @@
 
   - name: add copr-buildsys keys to copr user path
     action: copy src={{ item }} dest=/home/copr/cloud/ owner=copr group=copr mode=0600
-    with_fileglob: "{{ private }}/files/openstack/copr-copr/*"
+    with_fileglob:
+     - "{{ private }}/files/openstack/copr-copr/*"
 
   - name: setup privkey for copr user
     action: copy src="{{ private }}/files/copr/buildsys.priv" dest=/home/copr/.ssh/id_rsa owner=copr group=copr mode=600
@@ -144,14 +145,16 @@
 
   - name: put files into the files subdir off of provisioning
     action: copy src={{ item }} dest=/home/copr/provision/files/
-    with_fileglob: "{{ files }}/copr/provision/files/*"
+    with_fileglob:
+     - "{{ files }}/copr/provision/files/*"
     tags:
     - provision_config
 
   # ansible lacks a recurse - so we need this until then
   - name: put files into the files/mock subdir off of provisioning
     action: copy src={{ item }} dest=/home/copr/provision/files/mock
-    with_fileglob: "{{ files }}/copr/provision/files/mock/*"
+    with_fileglob:
+     - "{{ files }}/copr/provision/files/mock/*"
     tags:
     - provision_config
 

playbooks/sign_and_import.yml

@@ -32,28 +32,28 @@
   - name: make a directory where we store the rpms afterwards
     file: path={{ rpmdir }}-old state=directory
 
-
-
   - name: copy the source rpms to the SRPMS dir
     copy: src=${item} dest=${repodir}/SRPMS/
-    with_fileglob: ${rpmdir}/*.src.rpm
+    with_fileglob:
+     - "{{ rpmdir }}/*.src.rpm"
 
   - name: Run createrepo on the srpm dir
     command: createrepo --update ${repodir}/SRPMS/
 
   - name: move processed srpms out to ${rpmdir}-old
     command: /bin/mv ${item} ${rpmdir}-old/
-    with_fileglob: ${rpmdir}/*.src.rpm
-
-
+    with_fileglob:
+     - "{{ rpmdir }}/*.src.rpm"
 
   - name: copy the binary rpms to the x86_64 dir
     copy: src=${item} dest=${repodir}/x86_64/
-    with_fileglob: ${rpmdir}/*.rpm
+    with_fileglob:
+     - "{{ rpmdir }}/*.rpm"
 
   - name: Run createrepo on the x86_64 dir
     command: createrepo --update ${repodir}/x86_64/
 
   - name: move processed rpms out to ${rpmdir}-old
     command: /bin/mv ${item} ${rpmdir}-old/
-    with_fileglob: ${rpmdir}/*.rpm
+    with_fileglob:
+     - "{{ rpmdir }}/*.rpm"

roles/geoip/tasks/main.yml

@@ -7,7 +7,8 @@
 
 - name: push over the geoip db once
   copy: src={{ item }} dest=/usr/share/GeoIP/
-  with_fileglob: "{{ bigfiles }}/geoip/*.dat"
+  with_fileglob:
+   - "{{ bigfiles }}/geoip/*.dat"
 
 - name: geoip syncing script via cron
   copy: src=geoip_sync dest=/etc/cron.d/geoip_sync mode=0644