From 2276fbbe3d500bd8b52233215f0fa346cb88be69 Mon Sep 17 00:00:00 2001
From: Abhinandan Purkait <purkaitabhinandan@gmail.com>
Date: Wed, 18 Dec 2024 11:05:20 +0000
Subject: [PATCH] feat(helm chart): add value to modify ps retries

Signed-off-by: Abhinandan Purkait <purkaitabhinandan@gmail.com>
---
 chart/README.md                                      | 1 +
 chart/templates/mayastor/io/io-engine-daemonset.yaml | 1 +
 chart/values.yaml                                    | 2 ++
 3 files changed, 4 insertions(+)

diff --git a/chart/README.md b/chart/README.md
index 3fa4b11ff..790206878 100644
--- a/chart/README.md
+++ b/chart/README.md
@@ -176,6 +176,7 @@ This removes all the Kubernetes components associated with the chart and deletes
 | io_engine.&ZeroWidthSpace;nvme.&ZeroWidthSpace;ioTimeout | Timeout for IOs The default here is exaggerated for local disks, but we've observed that in shared virtual environments having a higher timeout value is beneficial. Please adjust this according to your hardware and needs. | `"110s"` |
 | io_engine.&ZeroWidthSpace;nvme.&ZeroWidthSpace;tcp.&ZeroWidthSpace;maxQueueDepth | You may need to increase this for a higher outstanding IOs per volume | `"32"` |
 | io_engine.&ZeroWidthSpace;priorityClassName | Set PriorityClass, overrides global | `""` |
+| io_engine.&ZeroWidthSpace;pstorRetries | Number of retries for pstor persistence before the volume target self shutdowns | `300` |
 | io_engine.&ZeroWidthSpace;resources.&ZeroWidthSpace;limits.&ZeroWidthSpace;cpu | Cpu limits for the io-engine | `""` |
 | io_engine.&ZeroWidthSpace;resources.&ZeroWidthSpace;limits.&ZeroWidthSpace;hugepages2Mi | Hugepage size available on the nodes | `"2Gi"` |
 | io_engine.&ZeroWidthSpace;resources.&ZeroWidthSpace;limits.&ZeroWidthSpace;memory | Memory limits for the io-engine | `"1Gi"` |
diff --git a/chart/templates/mayastor/io/io-engine-daemonset.yaml b/chart/templates/mayastor/io/io-engine-daemonset.yaml
index 56cddab48..eed3c5af4 100644
--- a/chart/templates/mayastor/io/io-engine-daemonset.yaml
+++ b/chart/templates/mayastor/io/io-engine-daemonset.yaml
@@ -111,6 +111,7 @@ spec:
         - "--reactor-freeze-detection"{{ end }}
         - "--tgt-crdt={{ .Values.io_engine.target.nvmf.hostCmdRetryDelay.crdt1 }}"{{ if .Values.eventing.enabled }}
         - "--events-url=nats://{{ .Release.Name }}-nats:4222"{{ end }}
+        - "--ps-retries={{ default 300 .Values.io_engine.pstorRetries }}"
         command:
         - io-engine
         securityContext:
diff --git a/chart/values.yaml b/chart/values.yaml
index b38c176b3..b2a75057e 100644
--- a/chart/values.yaml
+++ b/chart/values.yaml
@@ -466,6 +466,8 @@ io_engine:
   priorityClassName: ""
   # -- Runtime class to use. Defaults to cluster standard
   runtimeClassName: ""
+  # -- Number of retries for pstor persistence before the volume target self shutdowns
+  pstorRetries: 300
 
 etcd:
   # -- Disable when using an external etcd cluster.