Update fastpackage.yml

roles/jws/tasks/fastpackage.yml

@@ -1,19 +1,17 @@
 ---
-- name: Check arguments
-  ansible.builtin.assert:
-    that:
-      - package_name is defined
-    quiet: true
+- name: "Check if \"{{ packages_list | join(', ') }}\" packages are already installed" # noqa command-instead-of-module this runs faster
+  ansible.builtin.command: "rpm -q {{ packages_list | join(' ') }}"
+  register: rpm_info
+  changed_when: False
+  failed_when: False
 
-- name: "Install package"
-  block:
-    - name: "Test if package {{ package_name }} is already installed"
-      ansible.builtin.command: rpm -q {{ package_name }}
-      register: rpm_info
-      changed_when: rpm_info.failed
-  rescue:
-    - name: "Install package {{ package_name }} using yum if missing"
-      ansible.builtin.yum:
-        name: "{{ package_name }}"
-        state: installed
-      when: rpm_info.failed
+- name: "Add missing packages to the yum install list"
+  ansible.builtin.set_fact:
+    packages_to_install: "{{ packages_to_install | default([]) + rpm_info.stdout_lines | map('regex_findall', 'package (.+) is not installed$') | default([]) | flatten }}"
+
+- name: "Install packages: {{ packages_to_install }}"
+  become: True
+  ansible.builtin.yum:
+    name: "{{ packages_to_install }}"
+    state: present
+  when: packages_to_install | default([]) | length > 0

roles/jws/tasks/java_install.yml

@@ -11,7 +11,8 @@
     - name: "Install {{ jws_java_packages_el }}"
       ansible.builtin.include_tasks: fastpackage.yml
       vars:
-        package_name: "{{ jws_java_packages_el }}"
+        packages_list:
+          - "{{ jws_java_packages_el }}"
 
     - name: Determine JAVA_HOME for selected JVM RPM
       ansible.builtin.set_fact:

roles/jws/tasks/main.yml

@@ -36,16 +36,14 @@
 - name: "Install required dependencies"
   ansible.builtin.include_tasks: fastpackage.yml
   vars:
-    package_name: "{{ item }}"
-  loop: "{{ jws_dependencies_list }}"
+    packages_list: "{{ jws_dependencies_list }}"
 
 - name: "Install required dependencies for natives"
   ansible.builtin.include_tasks: fastpackage.yml
   vars:
-    package_name: "{{ item }}"
-  loop:
-    - openssl
-    - apr
+    packages_list:
+      - openssl
+      - apr
   when: jws_native
 
 - name: "Ensure tomcatjss rpm is not installed"

roles/jws/tasks/systemd/selinux.yml

@@ -20,7 +20,8 @@
     - name: Install selinux-policy-devel
       ansible.builtin.include_tasks: ../fastpackage.yml
       vars:
-        package_name: selinux-policy-devel
+        packages_list:
+          - selinux-policy-devel
 
     - name: Check if archive contains a selinux policy
       ansible.builtin.stat:

roles/jws/tasks/systemd/systemd.yml

@@ -11,11 +11,10 @@
 
 - name: "Ensure requirements for systemd"
   ansible.builtin.include_tasks: ../fastpackage.yml
-  loop_control:
-    loop_var: package_name
-  loop:
-    - systemd
-    - procps-ng
+  vars:
+    packages_list:
+      - systemd
+      - procps-ng
 
 - name: "Set required default for jws_service_conf if not provided."
   ansible.builtin.set_fact: