From 3e91c3a091441d029798ff7366950a1fb39776b2 Mon Sep 17 00:00:00 2001
From: "Ben D." <ben@noreply.localhost>
Date: Tue, 28 Apr 2026 10:26:32 -0700
Subject: [PATCH] Updated step 2 to prevent running when backup is current

---
 roles/pfsense_upgrade/tasks/carp.yml | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/roles/pfsense_upgrade/tasks/carp.yml b/roles/pfsense_upgrade/tasks/carp.yml
index 32e42db..857036d 100644
--- a/roles/pfsense_upgrade/tasks/carp.yml
+++ b/roles/pfsense_upgrade/tasks/carp.yml
@@ -60,7 +60,6 @@
   when:
     - ha_role == 'primary'
     - ha_peer is defined
-  ### Not altered ###
 
 - name: "[CARP/primary] Set peer version fact"
   ansible.builtin.set_fact:
@@ -68,7 +67,6 @@
   when:
     - ha_role == 'primary'
     - ha_peer is defined
-  ### Not altered ###
 
 - name: "[CARP/primary] Warn if backup peer is not on newer version"
   ansible.builtin.debug:
@@ -89,7 +87,7 @@
     - ha_role == 'primary'
     - ha_peer is defined
     - ha_peer_version == pfsense_current_version
-    #- perform_upgrade | bool
+    # No perform_upgrade condition - always end play when versions equal
 
 # --- Step 3: Verify backup peer is in MASTER CARP state (rewritten) ---
 - name: "[CARP/primary] Verify backup peer is MASTER for all CARP VIPs"