From ae79b54b92924bdea66567a55f2fa6b7fd7bc3d1 Mon Sep 17 00:00:00 2001
From: srikantpanda <srikant.panda@facets.cloud>
Date: Thu, 5 Feb 2026 13:02:57 +0530
Subject: [PATCH] updated storage class reclaim policy to include Valid
 options: Retain, Delete.

---
 deploy/helm/templates/storageclass.yaml | 12 ++++++------
 deploy/helm/values.yaml                 |  7 +++++++
 2 files changed, 13 insertions(+), 6 deletions(-)

diff --git a/deploy/helm/templates/storageclass.yaml b/deploy/helm/templates/storageclass.yaml
index 0a289387634..bf29386bf5a 100644
--- a/deploy/helm/templates/storageclass.yaml
+++ b/deploy/helm/templates/storageclass.yaml
@@ -14,7 +14,7 @@ parameters:
   type: {{ .Values.storageClass.provider.aws.volumeType }} # io2, io1, gp3, gp2 are all SSD variant
   "csi.storage.k8s.io/fstype": {{ .Values.storageClass.provider.aws.fsType | default "xfs" }}
 provisioner: ebs.csi.aws.com
-reclaimPolicy: Delete
+reclaimPolicy: {{ .Values.storageClass.reclaimPolicy | default "Delete" }}
 volumeBindingMode: WaitForFirstConsumer
 ---
 {{- else if eq $cloudProvider "gcp" }}
@@ -30,7 +30,7 @@ parameters:
   type: {{ .Values.storageClass.provider.gcp.volumeType }}
   csi.storage.k8s.io/fstype: {{ .Values.storageClass.provider.gcp.fsType | default "xfs" }}
 provisioner: pd.csi.storage.gke.io
-reclaimPolicy: Delete
+reclaimPolicy: {{ .Values.storageClass.reclaimPolicy | default "Delete" }}
 volumeBindingMode: WaitForFirstConsumer
 ---
 {{- else if eq $cloudProvider "aliyun" }}
@@ -46,7 +46,7 @@ parameters:
   fstype: {{ .Values.storageClass.provider.aliyun.fsType | default "xfs" }}
   type: {{ .Values.storageClass.provider.aliyun.volumeType }}
 provisioner: diskplugin.csi.alibabacloud.com
-reclaimPolicy: Delete
+reclaimPolicy: {{ .Values.storageClass.reclaimPolicy | default "Delete" }}
 volumeBindingMode: WaitForFirstConsumer
 ---
 {{- else if eq $cloudProvider "tencentCloud" }}
@@ -59,7 +59,7 @@ metadata:
 parameters:
   ## parameters references: https://cloud.tencent.com/document/product/457/44239, the fsType is not supported by tke.
   type: {{ .Values.storageClass.provider.tencentCloud.volumeType }}
-reclaimPolicy: Delete
+reclaimPolicy: {{ .Values.storageClass.reclaimPolicy | default "Delete" }}
 provisioner: com.tencent.cloud.csi.cbs
 volumeBindingMode: WaitForFirstConsumer
 ---
@@ -78,7 +78,7 @@ parameters:
   kind: {{ .Values.storageClass.provider.azure.volumeType }}
   skuName: Standard_LRS
 provisioner: kubernetes.io/azure-disk
-reclaimPolicy: Delete
+reclaimPolicy: {{ .Values.storageClass.reclaimPolicy | default "Delete" }}
 volumeBindingMode: WaitForFirstConsumer
 ---
 {{- else if eq $cloudProvider "huaweiCloud" }} # huawei cloud
@@ -95,7 +95,7 @@ parameters:
   everest.io/disk-volume-type: {{ .Values.storageClass.provider.huaweiCloud.volumeType }}
   everest.io/passthrough: "true"
 provisioner: everest-csi-provisioner
-reclaimPolicy: Delete
+reclaimPolicy: {{ .Values.storageClass.reclaimPolicy | default "Delete" }}
 volumeBindingMode: WaitForFirstConsumer
 allowVolumeExpansion: true
 ---
diff --git a/deploy/helm/values.yaml b/deploy/helm/values.yaml
index 0dad2ede45d..705e14c6d7d 100644
--- a/deploy/helm/values.yaml
+++ b/deploy/helm/values.yaml
@@ -511,6 +511,13 @@ storageClass:
   ## specified or generated, this value will be ignored.
   ##
   create: true
+  ## @param storageClass.reclaimPolicy -- Specifies the reclaim policy for the storage class.
+  ## Valid options: Retain, Delete. Default is Delete to preserve existing behaviour.
+  ## Set to Retain for production datastore workloads to protect PVCs from deletion.
+  ## Note: reclaimPolicy is inherited by PVs at creation time only — existing PVs must
+  ## be patched individually if this value is changed after provisioning.
+  ##
+  reclaimPolicy: Delete
   mountOptions:
   - noatime
   - nobarrier