Fix trivial ansible lint warnings

Remove no more used playbook, fix ansible-lint version used in the
github workflow, tune the yamllint to be compliant with Ansible
(remove warning about it reported by ansible-lint),
add global exception about role-name[path] as it require project folder
structure change, and last but not least fix some warnings.

Author: mpagot

ansible/playbooks/additional_fence_agent_tasks.yaml

@@ -11,15 +11,14 @@
     ptf_dir: "/tmp/ptf_dir"
 
   tasks:
-
     - name: Create directory for PTF installation
-      file:
+      ansible.builtin.file:
         path: "{{ ptf_dir }}"
         state: directory
         mode: '0755'
 
     - name: Convert comma-separated string to list
-      set_fact:
+      ansible.builtin.set_fact:
         az_blobs: "{{ ptf_files | split(',') }}"
       when: sas_token is defined
 
@@ -38,46 +37,49 @@
       with_items: "{{ az_blobs }}"
       when: sas_token is defined
 
-    - name: Download PTF files recursively with wget
-      command: "wget --no-directories --recursive --reject 'index.html*' --user={{ ptf_user }} --password={{ ptf_password }} --no-parent {{ ptf_url }}"
+    - name: Download PTF files recursively with wget  # noqa: command-instead-of-module
+      ansible.builtin.command: "wget --no-directories --recursive --reject 'index.html*' --user={{ ptf_user }} --password={{ ptf_password }} --no-parent {{ ptf_url }}"
       args:
         chdir: "{{ ptf_dir }}"
+      register: wget_ret
+      changed_when: wget_ret.rc == 0
       when:
         - sas_token is not defined
         - ptf_user is defined
         - ptf_password is defined
         - ptf_url is defined
 
     - name: List downloaded files
-      command: "ls -la"
+      ansible.builtin.command: "ls -la"
       args:
         chdir: "{{ ptf_dir }}"
+      changed_when: false
       register: download_list
 
     - name: Display downloaded files
-      debug:
+      ansible.builtin.debug:
         var: download_list.stdout_lines
 
     - name: Find downloaded RPM files
-      find:
+      ansible.builtin.find:
         paths: "{{ ptf_dir }}"
         patterns: "*.rpm"
       register: rpm_files
 
     - name: Display found RPM files
-      debug:
+      ansible.builtin.debug:
         var: rpm_files.files
 
     - name: Filter out src.rpm files
-      set_fact:
+      ansible.builtin.set_fact:
         filtered_rpm_files: "{{ rpm_files.files | selectattr('path', 'search', '^(?!.*src\\.rpm$).*') | list }}"
 
     - name: Display filtered RPM files
-      debug:
+      ansible.builtin.debug:
         var: filtered_rpm_files
 
     - name: Install PTF RPM packages
-      zypper:
+      community.general.zypper:
         name: "{{ item.path }}"
         state: present
         disable_gpg_check: true

ansible/playbooks/ptf_installation.yaml

@@ -1,12 +1,13 @@
 ---
-- hosts: hana
+- name: SAP Hana System Replication Hooks
+  hosts: hana
   remote_user: cloudadmin
   become: true
   become_user: root
 
   pre_tasks:
     - name: Source hana install variables
-      include_vars: ./vars/hana_vars.yaml
+      ansible.builtin.include_vars: ./vars/hana_vars.yaml
 
   vars:
     is_primary: "{{ ansible_play_hosts[0] == inventory_hostname }}"
@@ -26,22 +27,17 @@
           {{ __sapcontrol }}
           -nr {{ sap_hana_install_instance_number }}
           -function StopWait 600 10
+      register: stopwait
+      changed_when: stopwait.rc == 0  # Sucessfull output is like "\n24.06.2025 23:38:21\nStop\nOK\n\n24.06.2025 23:38:41\nStopWait\nOK"
 
-    - name: Start HANA Primary
+    - name: Start HANA
       ansible.builtin.command:  # StartWait will timeout after 600 seconds.
         cmd: >-
           {{ __sapcontrol }}
           -nr {{ sap_hana_install_instance_number }}
           -function StartWait 600 10
-      when: is_primary
-
-    - name: Start HANA Secondary
-      ansible.builtin.command:  # StartWait will timeout after 600 seconds.
-        cmd: >-
-          {{ __sapcontrol }}
-          -nr {{ sap_hana_install_instance_number }}
-          -function StartWait 600 10
-      when: not is_primary
+      register: startwait
+      changed_when: startwait.rc == 0  # Sucessfull output is like "\n24.06.2025 23:38:42\nStart\nOK\n\n24.06.2025 23:39:52\nStartWait\nOK"
 
   tasks:
     - name: Assert that required variables are defined

ansible/playbooks/sap-hana-system-replication-hooks.yaml

@@ -1,5 +1,6 @@
 ---
-- hosts: hana
+- name: SAP Hana System Replication
+  hosts: hana
   remote_user: cloudadmin
   become: true
   become_user: root
@@ -11,7 +12,7 @@
       when: ansible_play_hosts | length != 2
 
     - name: Load HANA vars
-      include_vars: ./vars/hana_vars.yaml
+      ansible.builtin.include_vars: ./vars/hana_vars.yaml
 
   vars:
     install_path: /var/lib/qedep
@@ -52,7 +53,7 @@
 
     - name: Execute hana system replication role
       ansible.builtin.include_role:
-        role: ../roles/sap_ha_install_hana_hsr
+        name: ../roles/sap_ha_install_hana_hsr
       when: hsr_configured == 'absent'
 
     - name: Write status file