Merge "Don't use loops with package task"

playbooks/roles/base-server/tasks/main.yaml

@@ -1,8 +1,7 @@
 - name: Install base packages
   package:
     state: present
-    name: '{{ item }}'
-  loop: '{{ base_packages }}'
+    name: "{{ base_packages }}"
 
 - name: Include OS-specific variables
   include_vars: "{{ lookup('first_found', params) }}"
@@ -15,8 +14,7 @@
 - name: Install distro specific packages
   package:
     state: present
-    name: '{{ item }}'
-  loop: '{{ distro_packages }}'
+    name: "{{ distro_packages }}"
 
 - name: Increase syslog message size in order to capture python tracebacks
   copy:

playbooks/roles/pip3/tasks/main.yaml

@@ -1,6 +1,5 @@
 - name: Install pip and virtualenv
   package:
-    name: "{{ item }}"
-  loop:
-    - python3-pip
-    - python3-virtualenv
+    name:
+      - python3-pip
+      - python3-virtualenv

roles/puppet-install/tasks/puppet-install/trusty.yaml

@@ -16,10 +16,9 @@
     - name: Install puppet packages
       package:
         state: present
-        name: '{{ item }}'
-      loop:
-        - puppet
-        - ruby
+        name:
+          - puppet
+          - ruby
     # wipe out templatedir so we don't get warnings
     - name: Remove templatedir
       lineinfile:
@@ -54,4 +53,4 @@
     name: puppet
     state: stopped
     enabled: no
-  become: yes
+  become: yes

roles/puppet-install/tasks/puppet-install/xenial.yaml

@@ -11,10 +11,9 @@
     - name: Install puppet packages
       package:
         state: present
-        name: '{{ item }}'
-      loop:
-        - puppet
-        - ruby
+        name:
+          - puppet
+          - ruby
 
     # wipe out templatedir so we don't get warnings
     - name: Remove templatedir