Merge "tuned - check packages before install"

releasenotes/notes/check-installed-packages-before-invoking-package-module-ede8b9c3e4944a0b.yaml

@@ -3,11 +3,11 @@ fixes:
   - |
     Deployment failure was observed when repository files are present, but the
     ctlplane is a non routed network. The node cannot connect to repositories
-    before the network is configured in this case. The tripleo-bootstrap role
-    uses the ansible built-in ``package`` which fail despite the fact that the
-    packages are already installed in case it cannot connect to repositories.
+    before the network is configured in this case. Some roles use the ansible
+    built-in ``package`` which fail despite the fact that the packages are
+    already installed in case it cannot connect to repositories.
 
-    The tripleo-bootstrap role will now inspect the installed packages to
-    check wheater or not packages are already installed. When packages are
-    installed, the ``package`` tasks are skipped. See bug:
+    The roles tripleo-bootstrap and tuned will now inspect the installed
+    packages to check wheater or not packages are already installed. When
+    packages are installed, the ``package`` tasks are skipped. See bug:
     `1959731 <https://bugs.launchpad.net/tripleo/+bug/1959731>`_.

tripleo_ansible/roles/tuned/tasks/tuned_install.yml

@@ -14,6 +14,12 @@
 # License for the specific language governing permissions and limitations
 # under the License.
 
+- name: Check tuned package is installed
+  command: "rpm -q --whatprovides {{ tuned_system_packages }}"
+  args:
+    warn: false
+  register: rpm_query_result
+  failed_when: false
 
 - name: Install tuned
   become: true
@@ -21,6 +27,8 @@
     name: "{{ tuned_system_packages }}"
     state: present
   register: _tuned_install_result
+  when:
+    - rpm_query_result.rc > 0
 
 - name: Restart tuned
   become: true