From 1520919b5b4aeac0b7bd5933739435739b8168b2 Mon Sep 17 00:00:00 2001 From: Rob Dobozy Date: Tue, 14 Jan 2025 11:38:38 +0000 Subject: [PATCH] sap_swpm: Change shell to command module for process check and set changed_when to false to fix lint errors --- roles/sap_swpm/tasks/post_install/sum_push_to_finish.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/roles/sap_swpm/tasks/post_install/sum_push_to_finish.yml b/roles/sap_swpm/tasks/post_install/sum_push_to_finish.yml index 20f963bd..24e416a2 100644 --- a/roles/sap_swpm/tasks/post_install/sum_push_to_finish.yml +++ b/roles/sap_swpm/tasks/post_install/sum_push_to_finish.yml @@ -6,12 +6,13 @@ # Check if the SUMup process is running, give it 5 minutes to start - name: Check if SAPup_real process is running (wait for 5 minutes until it starts) - ansible.builtin.shell: pgrep -c -u "{{ sap_swpm_sid | lower }}adm" -f SAPup_real + ansible.builtin.command: pgrep -c -u "{{ sap_swpm_sid | lower }}adm" -f SAPup_real register: _sapup_process retries: 5 delay: 60 until: _sapup_process.rc == 0 and _sapup_process.stdout|int > 0 failed_when: _sapup_process.rc != 0 + changed_when: false - name: Print SUM monitoring and action URLs ansible.builtin.debug: