From 1230e6b5a5dab9b3dc36ebb349a70d748d9d1883 Mon Sep 17 00:00:00 2001 From: hemanthsavasere Date: Sun, 25 Jan 2026 05:57:39 +0000 Subject: [PATCH 1/2] Helm Chart Component Configuration Isolation --- helm/templates/sts-coordinator.yaml | 24 ++++++++++---------- helm/templates/sts-tablet.yaml | 22 +++++++++--------- helm/values.yaml | 35 +++++++++++++++++++++++------ 3 files changed, 51 insertions(+), 30 deletions(-) diff --git a/helm/templates/sts-coordinator.yaml b/helm/templates/sts-coordinator.yaml index 3caa783957..9db50ab7c2 100644 --- a/helm/templates/sts-coordinator.yaml +++ b/helm/templates/sts-coordinator.yaml @@ -24,7 +24,7 @@ metadata: {{- include "fluss.labels" . | nindent 4 }} spec: serviceName: coordinator-server-hs - replicas: 1 # can only be 1 for now + replicas: {{ .Values.coordinatorServer.replicas }} selector: matchLabels: {{- include "fluss.selectorLabels" . | nindent 6 }} @@ -73,21 +73,21 @@ spec: bin/coordinator-server.sh start-foreground livenessProbe: - failureThreshold: 100 - timeoutSeconds: 1 - initialDelaySeconds: 10 - periodSeconds: 3 + failureThreshold: {{ .Values.coordinatorServer.livenessProbe.failureThreshold }} + timeoutSeconds: {{ .Values.coordinatorServer.livenessProbe.timeoutSeconds }} + initialDelaySeconds: {{ .Values.coordinatorServer.livenessProbe.initialDelaySeconds }} + periodSeconds: {{ .Values.coordinatorServer.livenessProbe.periodSeconds }} tcpSocket: - port: {{.Values.appConfig.externalPort}} + port: {{ .Values.appConfig.externalPort }} readinessProbe: - failureThreshold: 100 - timeoutSeconds: 1 - initialDelaySeconds: 10 - periodSeconds: 3 + failureThreshold: {{ .Values.coordinatorServer.readinessProbe.failureThreshold }} + timeoutSeconds: {{ .Values.coordinatorServer.readinessProbe.timeoutSeconds }} + initialDelaySeconds: {{ .Values.coordinatorServer.readinessProbe.initialDelaySeconds }} + periodSeconds: {{ .Values.coordinatorServer.readinessProbe.periodSeconds }} tcpSocket: - port: {{.Values.appConfig.externalPort}} + port: {{ .Values.appConfig.externalPort }} resources: - {{- toYaml .Values.resources.tabletServer | nindent 12 }} + {{- toYaml .Values.resources.coordinatorServer | nindent 12 }} volumeMounts: - name: fluss-conf mountPath: /opt/conf diff --git a/helm/templates/sts-tablet.yaml b/helm/templates/sts-tablet.yaml index f13dce9b1b..24470961ee 100644 --- a/helm/templates/sts-tablet.yaml +++ b/helm/templates/sts-tablet.yaml @@ -24,7 +24,7 @@ metadata: {{- include "fluss.labels" . | nindent 4 }} spec: serviceName: tablet-server-hs - replicas: 3 + replicas: {{ .Values.tabletServer.replicas }} selector: matchLabels: {{- include "fluss.selectorLabels" . | nindent 6 }} @@ -68,19 +68,19 @@ spec: echo "advertised.listeners: CLIENT://${POD_NAME}.tablet-server-hs.${POD_NAMESPACE}.svc.cluster.local:{{ .Values.appConfig.externalPort }}" >> $FLUSS_HOME/conf/server.yaml && \ bin/tablet-server.sh start-foreground livenessProbe: - failureThreshold: 100 - timeoutSeconds: 1 - initialDelaySeconds: 10 - periodSeconds: 3 + failureThreshold: {{ .Values.tabletServer.livenessProbe.failureThreshold }} + timeoutSeconds: {{ .Values.tabletServer.livenessProbe.timeoutSeconds }} + initialDelaySeconds: {{ .Values.tabletServer.livenessProbe.initialDelaySeconds }} + periodSeconds: {{ .Values.tabletServer.livenessProbe.periodSeconds }} tcpSocket: - port: {{.Values.appConfig.externalPort}} + port: {{ .Values.appConfig.externalPort }} readinessProbe: - failureThreshold: 100 - timeoutSeconds: 1 - initialDelaySeconds: 10 - periodSeconds: 3 + failureThreshold: {{ .Values.tabletServer.readinessProbe.failureThreshold }} + timeoutSeconds: {{ .Values.tabletServer.readinessProbe.timeoutSeconds }} + initialDelaySeconds: {{ .Values.tabletServer.readinessProbe.initialDelaySeconds }} + periodSeconds: {{ .Values.tabletServer.readinessProbe.periodSeconds }} tcpSocket: - port: {{.Values.appConfig.externalPort}} + port: {{ .Values.appConfig.externalPort }} resources: {{- toYaml .Values.resources.tabletServer | nindent 12 }} volumeMounts: diff --git a/helm/values.yaml b/helm/values.yaml index 9cf4b93293..6c0b03b1ba 100644 --- a/helm/values.yaml +++ b/helm/values.yaml @@ -31,6 +31,31 @@ appConfig: internalPort: 9123 externalPort: 9124 +coordinatorServer: + replicas: 1 + livenessProbe: + failureThreshold: 100 + timeoutSeconds: 1 + initialDelaySeconds: 10 + periodSeconds: 3 + readinessProbe: + failureThreshold: 100 + timeoutSeconds: 1 + initialDelaySeconds: 10 + periodSeconds: 3 + +tabletServer: + replicas: 3 + livenessProbe: + failureThreshold: 100 + timeoutSeconds: 1 + initialDelaySeconds: 10 + periodSeconds: 3 + readinessProbe: + failureThreshold: 100 + timeoutSeconds: 1 + initialDelaySeconds: 10 + periodSeconds: 3 # Fluss server configuration options configurationOverrides: @@ -47,19 +72,15 @@ persistence: size: 1Gi storageClass: -resources: {} - # We usually recommend not to specify default resources and to leave this as a conscious - # choice for the user. This also increases chances charts run on environments with little - # resources, such as Minikube. If you do want to specify resources, uncomment the following - # lines, adjust them as necessary, and remove the curly braces after 'resources:'. - # coordinatorServer: +resources: + coordinatorServer: {} # limits: # cpu: 100m # memory: 128Mi # requests: # cpu: 100m # memory: 128Mi - # tabletServer: + tabletServer: {} # requests: # cpu: 100m # memory: 128Mi From c5bdea6f89b5229007e90ec1e48c32ae198f4b49 Mon Sep 17 00:00:00 2001 From: hemanthsavasere Date: Sun, 25 Jan 2026 13:18:23 +0000 Subject: [PATCH 2/2] Retrigger CI tests