From bda19143a62c37d194d8504e16d78df1b956ffa0 Mon Sep 17 00:00:00 2001
From: Chao Zhang <charles.zhangzs@gmail.com>
Date: Thu, 29 Aug 2024 14:19:41 +0200
Subject: [PATCH] make each probe configurable

---
 charts/tezos/templates/_containers.tpl | 14 +++++++++-----
 charts/tezos/values.yaml               | 24 +++++++++++++++---------
 2 files changed, 24 insertions(+), 14 deletions(-)

diff --git a/charts/tezos/templates/_containers.tpl b/charts/tezos/templates/_containers.tpl
index 5496933..b91195a 100644
--- a/charts/tezos/templates/_containers.tpl
+++ b/charts/tezos/templates/_containers.tpl
@@ -171,24 +171,28 @@
       name: tezos-net
     - containerPort: 9932
       name: metrics
-    {{- if or (not (hasKey $.node_vals "bootstrapped_probe")) $.node_vals.bootstrapped_probe }}
+    {{- if $.node_vals.bootstrapped_startup_probe }}
   startupProbe:
     httpGet:
       path: /is_synced
       port: 31732
     failureThreshold: 180
     periodSeconds: 10
-  readinessProbe:
+    {{- end }}
+    {{- if $.node_vals.bootstrapped_liveness_probe }}
+  livenessProbe:
     httpGet:
       path: /is_synced
       port: 31732
-    successThreshold: 1
+    failureThreshold: 30
     periodSeconds: 10
-  livenessProbe:
+    {{- end  }}
+    {{- if or (not (hasKey $.node_vals "bootstrapped_readiness_probe")) $.node_vals.bootstrapped_readiness_probe }}
+  readinessProbe:
     httpGet:
       path: /is_synced
       port: 31732
-    failureThreshold: 30
+    successThreshold: 1
     periodSeconds: 10
     {{- else if or (not (hasKey $.node_vals "rpc_readiness_probe")) $.node_vals.rpc_readiness_probe }}
   readinessProbe:
diff --git a/charts/tezos/values.yaml b/charts/tezos/values.yaml
index 099f414..c94d5af 100644
--- a/charts/tezos/values.yaml
+++ b/charts/tezos/values.yaml
@@ -153,15 +153,21 @@ accounts: {}
 # - `rpc_readiness_probe`: Attach a probe to the node. The probe checks whether
 #                          the RPC service is responsive, which should always be the
 #                          case. Defaults to true.
-# - `bootstrapped_probe`: Adds startup/readiness/liveness probe to check checks
-#                         whether the most recent block is less than 600 seconds old.
-#                         Overrides `rpc_readiness_probe`. Defaults to True.
-#                         True is good for RPC nodes, private nodes, and
-#                         self-contained private chains.
-#                         Recommended to set to False when bootstrapping a new
-#                         chain with external bakers, such as a new test chain.
-#                         Otherwise, the chain may become unreachable externally
-#                         while waiting for other nodes to come online.
+# - `bootstrapped_readiness_probe`: Checks whether the most recent block is less than
+#                                   600 seconds old.
+#                                   Overrides `rpc_readiness_probe`. Defaults to True.
+#                                   True is good for RPC nodes, private nodes, and
+#                                   self-contained private chains.
+#                                   Recommended to set to False when bootstrapping a new
+#                                   chain with external bakers, such as a new test chain.
+#                                   Otherwise, the chain may become unreachable externally
+#                                   while waiting for other nodes to come online.
+# - `bootstrapped_startup_probe`: Adds additional startup probe to check checks
+#                                 whether the most recent block is less than 600 seconds old.
+#                                 Defaults to False.
+# - `bootstrapped_liveness_probe`: Adds additional liveness probe to check checks
+#                                 whether the most recent block is less than 600 seconds old.
+#                                 Defaults to False.
 # - `instances`: A list of nodes to fire up, each is a dictionary defining:
 #    - `bake_using_accounts`: List of account names that should be used for baking.
 #    - `authorized_keys`: List of account names that should be used as keys to