From 8694773593ccc55814555954fedb4f7f2ae1d092 Mon Sep 17 00:00:00 2001
From: Nikholas Pcenicni <82239765+nikpcenicni@users.noreply.github.com>
Date: Wed, 13 May 2026 17:40:00 -0400
Subject: [PATCH] Disable pre-upgrade job in Longhorn values.yaml to prevent
 failures during Argo CD syncs when Longhorn is already healthy, ensuring
 cleaner deployments.

---
 clusters/noble/bootstrap/longhorn/values.yaml | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/clusters/noble/bootstrap/longhorn/values.yaml b/clusters/noble/bootstrap/longhorn/values.yaml
index 7501ad4..714c137 100644
--- a/clusters/noble/bootstrap/longhorn/values.yaml
+++ b/clusters/noble/bootstrap/longhorn/values.yaml
@@ -29,6 +29,9 @@ ingress:
   annotations:
     cert-manager.io/cluster-issuer: letsencrypt-prod
 
-# Pre-upgrade Job: keep enabled for normal Helm upgrades (disable only if GitOps sync fights the Job).
+# Pre-upgrade Job disabled: Argo CD triggers this hook on every sync, and when Longhorn is already
+# healthy the job still fails under Argo (hook lifecycle differs from `helm upgrade`).
+# For a real chart version bump, temporarily set jobEnabled: true, run via Ansible helm upgrade, then
+# set it back to false so Argo syncs stay clean.
 preUpgradeChecker:
-  jobEnabled: true
+  jobEnabled: false