[heartex/label-studio] Add missing readiness probe for app deployment & fix nginx readinessProbe reference (#60)

Author: elohmeier

heartex/label-studio/CHANGELOG.md

@@ -1,4 +1,9 @@
 # Changelog
+## 1.9.5
+### Fixes
+* Add missing readiness probe for app deployment app container.
+* Fix nginx container readiness probe reference in app deployment.
+
 ## 1.9.3
 ### Improvements
 * Support to omit secrets in downstream values.
@@ -28,7 +33,7 @@
 ## 1.7.0
 ### Improvements
 * Explicitly set dnsPolicy, shareProcessNamespace and enableServiceLinks.
-* Set shareProcessNamespace and enableServiceLinks to `false`. 
+* Set shareProcessNamespace and enableServiceLinks to `false`.
 
 ## 1.6.0
 ### Improvements

heartex/label-studio/Chart.yaml

@@ -5,7 +5,7 @@ home: https://labelstud.io/
 type: application
 icon: https://raw.githubusercontent.com/heartexlabs/label-studio/master/images/logo.png
 # Chart version
-version: 1.9.4
+version: 1.9.5
 # Label Studio release version
 appVersion: "1.16.0"
 kubeVersion: ">= 1.14.0-0"

heartex/label-studio/templates/app-deployment.yaml

@@ -146,6 +146,17 @@ spec:
             - name: CMD_WRAPPER
               value: {{ coalesce $.Values.app.cmdWrapper $.Values.global.cmdWrapper }}
             {{- end }}
+          {{- if .Values.app.readinessProbe.enabled }}
+          readinessProbe:
+            httpGet:
+              path: "{{ if eq .Values.app.contextPath "/" }}{{ .Values.app.readinessProbe.path }}{{ else }}{{ .Values.app.contextPath }}{{ .Values.app.readinessProbe.path }}{{ end }}"
+              port: 8000
+            failureThreshold: {{ .Values.app.readinessProbe.failureThreshold }}
+            initialDelaySeconds: {{ .Values.app.readinessProbe.initialDelaySeconds }}
+            periodSeconds: {{ .Values.app.readinessProbe.periodSeconds }}
+            successThreshold: {{ .Values.app.readinessProbe.successThreshold }}
+            timeoutSeconds: {{ .Values.app.readinessProbe.timeoutSeconds }}
+          {{- end }}
           {{- if .Values.app.livenessProbe.enabled }}
           livenessProbe:
             httpGet:
@@ -234,11 +245,11 @@ spec:
             httpGet:
               path: "{{ if eq .Values.app.contextPath "/" }}{{ .Values.app.nginx.readinessProbe.path }}{{ else }}{{ .Values.app.contextPath }}{{ .Values.app.nginx.readinessProbe.path }}{{ end }}"
               port: 8085
-            failureThreshold: {{ .Values.app.readinessProbe.failureThreshold }}
-            initialDelaySeconds: {{ .Values.app.readinessProbe.initialDelaySeconds }}
-            periodSeconds: {{ .Values.app.readinessProbe.periodSeconds }}
-            successThreshold: {{ .Values.app.readinessProbe.successThreshold }}
-            timeoutSeconds: {{ .Values.app.readinessProbe.timeoutSeconds }}
+            failureThreshold: {{ .Values.app.nginx.readinessProbe.failureThreshold }}
+            initialDelaySeconds: {{ .Values.app.nginx.readinessProbe.initialDelaySeconds }}
+            periodSeconds: {{ .Values.app.nginx.readinessProbe.periodSeconds }}
+            successThreshold: {{ .Values.app.nginx.readinessProbe.successThreshold }}
+            timeoutSeconds: {{ .Values.app.nginx.readinessProbe.timeoutSeconds }}
           {{- end }}
           {{- if .Values.app.nginx.livenessProbe.enabled }}
           livenessProbe: