[castai-db-optimizer] remove readiness probe (#705)

castai · Feb 5, 2025 · b33d2f2 · b33d2f2
1 parent 3fcbb67
commit b33d2f2
Show file tree

Hide file tree

Showing 5 changed files with 2 additions and 19 deletions.
diff --git a/charts/castai-db-optimizer/Chart.yaml b/charts/castai-db-optimizer/Chart.yaml
@@ -2,4 +2,4 @@ apiVersion: v2
 name: castai-db-optimizer
 description: CAST AI database cache deployment.
 type: application
-version: 0.8.0
+version: 0.9.0
diff --git a/charts/castai-db-optimizer/README.md b/charts/castai-db-optimizer/README.md
@@ -1,6 +1,6 @@
 # castai-db-optimizer
 
-![Version: 0.8.0](https://img.shields.io/badge/Version-0.8.0-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square)
+![Version: 0.9.0](https://img.shields.io/badge/Version-0.9.0-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square)
 
 CAST AI database cache deployment.
 
@@ -22,7 +22,6 @@ CAST AI database cache deployment.
 | proxy.dnsLookupFamily | string | `"V4_PREFERRED"` | DNS lookup mode when communicating to outside. will prioritize IPV4 addresses. change to V6_ONLY to use v6 addresses instead. |
 | proxy.drainTimeSeconds | int | `60` | Default drain time in seconds. |
 | proxy.logLevel | string | `"filter:info"` | Default proxy log level. |
-| proxy.readinessProbeEnabled | bool | `true` | Ensure proxy has retrieved initial cache configuration before accepting connections. |
 | proxyImage.pullPolicy | string | `"IfNotPresent"` |  |
 | proxyImage.repository | string | `"us-docker.pkg.dev/castai-hub/library/dbo-proxy"` |  |
 | proxyImage.tag | string | `""` |  |

diff --git a/charts/castai-db-optimizer/ci/test-values.yaml b/charts/castai-db-optimizer/ci/test-values.yaml
@@ -7,9 +7,6 @@ resources:
   proxy:
     ephemeralStorage: 5Mi
 
-proxy:
-  readinessProbeEnabled: false
-
 podLabels:
   podLabel: label
 

diff --git a/charts/castai-db-optimizer/templates/deployment.yaml b/charts/castai-db-optimizer/templates/deployment.yaml
@@ -136,17 +136,6 @@ spec:
               exec:
                 command: ["sleep", {{ .Values.proxy.drainTimeSeconds | quote }} ]
           {{- end}}
-          {{- if .Values.proxy.readinessProbeEnabled }}
-          readinessProbe:
-            periodSeconds: 3
-            exec:
-              command:
-              - /bin/bash
-              - '-c'
-              - |-
-                 health=$(curl --silent localhost:9901/clusters | grep {{ .Values.apiURL }} | grep -c health_flags::healthy);
-                 if [[ $health -ne 1 ]]; then echo "Initial cache configuration wasn't pulled yet" && exit 1; fi
-          {{- end}}
           securityContext:
             capabilities:
               drop:

diff --git a/charts/castai-db-optimizer/values.yaml b/charts/castai-db-optimizer/values.yaml
@@ -46,8 +46,6 @@ upstreamPostgresPort: 5432
 proxy:
   # -- Default proxy log level.
   logLevel: "filter:info"
-  # -- Ensure proxy has retrieved initial cache configuration before accepting connections.
-  readinessProbeEnabled: true
   # -- Number of parallel processing streams. This needs to be balanced with cpu resoures for proxy and QP.
   concurrency: 12
   # -- DNS lookup mode when communicating to outside. will prioritize IPV4 addresses. change to V6_ONLY to use v6 addresses instead.