fix(ui): Change UI probes settings to scale 1 new pod (#247)

* fix(ui): Change UI probes settings to scale 1 new pod * Helm-Docs update --------- Co-authored-by: Marcin Jasion <[email protected]>
iterative · Nov 28, 2023 · 47e8f0e · 47e8f0e
1 parent c742dad
commit 47e8f0e
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 8 deletions.
diff --git a/charts/studio/Chart.yaml b/charts/studio/Chart.yaml
@@ -2,7 +2,7 @@ apiVersion: v2
 name: studio
 description: A Helm chart for Kubernetes
 type: application
-version: 0.7.16
+version: 0.7.17
 appVersion: "v2.57.0"
 maintainers:
   - name: iterative

diff --git a/charts/studio/README.md b/charts/studio/README.md
@@ -1,6 +1,6 @@
 # studio
 
-![Version: 0.7.16](https://img.shields.io/badge/Version-0.7.16-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: v2.57.0](https://img.shields.io/badge/AppVersion-v2.57.0-informational?style=flat-square)
+![Version: 0.7.17](https://img.shields.io/badge/Version-0.7.17-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: v2.57.0](https://img.shields.io/badge/AppVersion-v2.57.0-informational?style=flat-square)
 
 A Helm chart for Kubernetes
 
@@ -183,7 +183,7 @@ A Helm chart for Kubernetes
 | studioLeo.service.port | int | `8181` | Leo service port |
 | studioLeo.service.type | string | `"ClusterIP"` | Leo service type |
 | studioLeo.tolerations | list | `[]` | Leo tolerations configuration |
-| studioUi | object | `{"affinity":{},"autoscaling":{"enabled":false,"maxReplicas":5,"minReplicas":1,"targetCPUUtilizationPercentage":80},"envFromSecret":"","envVars":{},"image":{"pullPolicy":"IfNotPresent","repository":"docker.iterative.ai/studio-frontend"},"nodeSelector":{},"podAnnotations":{},"podSecurityContext":{},"replicaCount":1,"resources":{"limits":{"cpu":"1000m","memory":"2Gi"},"requests":{"cpu":"500m","memory":"1Gi"}},"securityContext":{},"service":{"port":3000,"type":"ClusterIP"},"strategy":{"rollingUpdate":{"maxSurge":"50%","maxUnavailable":0}},"tolerations":[]}` | Studio UI settings group |
+| studioUi | object | `{"affinity":{},"autoscaling":{"enabled":false,"maxReplicas":5,"minReplicas":1,"targetCPUUtilizationPercentage":80},"envFromSecret":"","envVars":{},"image":{"pullPolicy":"IfNotPresent","repository":"docker.iterative.ai/studio-frontend"},"nodeSelector":{},"podAnnotations":{},"podSecurityContext":{},"replicaCount":1,"resources":{"limits":{"cpu":"1000m","memory":"2Gi"},"requests":{"cpu":"500m","memory":"1Gi"}},"securityContext":{},"service":{"port":3000,"type":"ClusterIP"},"strategy":{"rollingUpdate":{"maxSurge":1,"maxUnavailable":0}},"tolerations":[]}` | Studio UI settings group |
 | studioUi.envFromSecret | string | `""` | The name of an existing Secret that contains sensitive environment variables passed to UI pods. |
 | studioUi.envVars | object | `{}` | Additional environment variables for ui pods |
 | studioWorker | object | `{"affinity":{},"autoscaling":{"enabled":false,"maxReplicas":5,"minReplicas":1,"targetCPUUtilizationPercentage":80},"envFromSecret":"","envVars":{},"image":{"pullPolicy":"IfNotPresent","repository":"docker.iterative.ai/studio-backend"},"logLevel":"info","nodeSelector":{},"podAnnotations":{},"podSecurityContext":{},"replicaCount":1,"resources":{"limits":{"cpu":"1000m","memory":"1Gi"},"requests":{"cpu":"500m","memory":"512Mi"}},"securityContext":{},"serviceAccount":{"annotations":{},"create":false,"name":""},"strategy":{"rollingUpdate":{"maxSurge":"50%","maxUnavailable":0}},"tolerations":[]}` | Studio worker settings group |

diff --git a/charts/studio/templates/deployment-studio-ui.yaml b/charts/studio/templates/deployment-studio-ui.yaml
@@ -8,7 +8,7 @@ spec:
   {{- if not .Values.studioUi.autoscaling.enabled }}
   replicas: {{ .Values.studioUi.replicaCount }}
   {{- end }}
-  {{- with .Values.studioBackend.strategy }}
+  {{- with .Values.studioUi.strategy }}
   strategy:
     {{- toYaml . | nindent 4 }}
   {{- end }}
@@ -49,8 +49,8 @@ spec:
             exec:
               command:
                 - /app/bin/startup-probe.sh
-            initialDelaySeconds: 2
-            periodSeconds: 1
+            initialDelaySeconds: 5
+            periodSeconds: 2
             failureThreshold: 60
           livenessProbe:
             tcpSocket:
@@ -66,7 +66,8 @@ spec:
             {{- end }}
               port: 3000
             initialDelaySeconds: 5
-            periodSeconds: 25
+            periodSeconds: 10
+            successThreshold: 3
             timeoutSeconds: 10
           envFrom:
             - configMapRef:

diff --git a/charts/studio/values.yaml b/charts/studio/values.yaml
@@ -384,7 +384,7 @@ studioUi:
   strategy:
     rollingUpdate:
       maxUnavailable: 0
-      maxSurge: 50%
+      maxSurge: 1
 
   autoscaling:
     enabled: false