From 765b4e7c05c9a27a1c1623b702e195d23b96a420 Mon Sep 17 00:00:00 2001
From: sebas_correa <sebastian.correa@nullplatform.io>
Date: Thu, 29 Jan 2026 18:10:53 -0300
Subject: [PATCH] feat(charts-base): delete hooks

---
 charts/base/templates/finalizer.yaml | 31 ----------------------------
 charts/base/templates/gateways.yaml  |  5 -----
 charts/base/templates/namespace.yaml |  3 ---
 3 files changed, 39 deletions(-)
 delete mode 100644 charts/base/templates/finalizer.yaml

diff --git a/charts/base/templates/finalizer.yaml b/charts/base/templates/finalizer.yaml
deleted file mode 100644
index 4e074e3..0000000
--- a/charts/base/templates/finalizer.yaml
+++ /dev/null
@@ -1,31 +0,0 @@
-apiVersion: batch/v1
-kind: Job
-metadata:
-  name: {{ .Release.Name }}-finalizer
-  annotations:
-    "helm.sh/hook": pre-delete
-    "helm.sh/hook-weight": "-5"
-spec:
-  template:
-    metadata:
-      name: {{ .Release.Name }}-finalizer
-    spec:
-      restartPolicy: Never
-      containers:
-        - name: blocker
-          image: alpine:latest
-          command:
-            - /bin/sh
-            - -c
-            - |
-              echo "⛔ DELETION BLOCKED ⛔"
-              echo "========================================================"
-              echo "Please contact Nullplatform team in order to uninstall the chart: {{ .Release.Name }}"
-              echo "========================================================"
-
-              # This job will remain running indefinitely, preventing the pre-delete hook from completing
-              # which in turn prevents Helm from deleting the release
-              while true; do
-                echo "Chart deletion blocked for release: {{ .Release.Name }}. Please contact Nullplatform team."
-                sleep 3600
-              done
diff --git a/charts/base/templates/gateways.yaml b/charts/base/templates/gateways.yaml
index bad96d4..89c99cb 100644
--- a/charts/base/templates/gateways.yaml
+++ b/charts/base/templates/gateways.yaml
@@ -12,9 +12,6 @@ metadata:
   name: {{ .Values.namespaces.gateway }}
   labels:
     name: {{ .Values.namespaces.gateway }}
-  annotations:
-    "helm.sh/hook": pre-install, pre-upgrade
-    "helm.sh/resource-policy": "keep"
 {{- end }}
 ---
 {{- if and .Values.gateways.enabled .Values.gateway.internal.enabled }}
@@ -89,7 +86,6 @@ metadata:
     {{- /* Default annotations for other providers */ -}}
     service.beta.kubernetes.io/port_443_health-probe_protocol: "tcp"
     {{- end }}
-    "helm.sh/resource-policy": "keep"
 spec:
   gatewayClassName: istio
   {{- with .Values.gateway.internal.addresses }}
@@ -228,7 +224,6 @@ metadata:
     {{- /* Default annotations for other providers */ -}}
     service.beta.kubernetes.io/port_443_health-probe_protocol: "tcp"
     {{- end }}
-    "helm.sh/resource-policy": "keep"
 spec:
   gatewayClassName: istio
   {{- with .Values.gateway.public.addresses }}
diff --git a/charts/base/templates/namespace.yaml b/charts/base/templates/namespace.yaml
index f29af6c..231394e 100644
--- a/charts/base/templates/namespace.yaml
+++ b/charts/base/templates/namespace.yaml
@@ -6,7 +6,6 @@ metadata:
   labels:
     name: {{ .Values.namespaces.nullplatformTools }}
   annotations:
-    "helm.sh/hook": pre-install, pre-upgrade
     openshift.io/cluster-monitoring: "true"
 {{- end }}
 ---
@@ -17,7 +16,5 @@ metadata:
   name: {{ .Values.namespaces.nullplatformApplications }}
   labels:
     name: {{ .Values.namespaces.nullplatformApplications }}
-  annotations:
-    "helm.sh/hook": pre-install, pre-upgrade
 {{- end }}
 ---