From 7bba240f6e4fb89be38b5d7ff3c76284fe5ed7ee Mon Sep 17 00:00:00 2001
From: Danny <danny@vanderschaft.net>
Date: Sat, 22 Nov 2025 20:37:40 +0100
Subject: [PATCH] when clauses in all package installers

---
 roles/pips/tasks/nagios_tasks.yml | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/roles/pips/tasks/nagios_tasks.yml b/roles/pips/tasks/nagios_tasks.yml
index 30ed08e..961f47a 100644
--- a/roles/pips/tasks/nagios_tasks.yml
+++ b/roles/pips/tasks/nagios_tasks.yml
@@ -35,7 +35,7 @@
     state: present
   become: true
   register: nrpe_install_result
-  when: "{{ ansible_facts['os_family'] }} == 'Debian' "
+  when: "{{ ansible_facts['os_family']  == 'Debian' and not nrpe_installed }}"
 
 - name: Ensure NRPE is installed (RedHat/CentOS)
   yum:
@@ -43,7 +43,7 @@
     state: present
   become: yes
   register: nrpe_install_result
-  when: "{{ ansible_facts['os_family'] }} == 'RedHat' "
+  when: "{{ ansible_facts['os_family']  == 'RedHat' and not nrpe_installed }}"
 
 - name: Ensure NRPE is installed (FreeBSD)
   ansible.builtin.pkgng:
@@ -53,6 +53,7 @@
   register: nrpe_install_result
   when:
     - ansible_facts['os_family'] == 'FreeBSD'
+    - not nrpe_installed
 
 
 - name: Set psutil_installed fact if any package name contains "psutil"