This is an automated email from the ASF dual-hosted git repository.

hanahmily pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/skywalking-banyandb-helm.git


The following commit(s) were added to refs/heads/master by this push:
     new 83db081  fix wrong format of backup/lifecycle/fodc resource (#49)
83db081 is described below

commit 83db0812ed36cdc6ecc09f4563354e3bb5bdf4f4
Author: mrproliu <[email protected]>
AuthorDate: Tue Jan 27 10:34:11 2026 +0800

    fix wrong format of backup/lifecycle/fodc resource (#49)
---
 chart/templates/cluster_data_statefulset.yaml      | 39 ++++++++++++++++++++--
 chart/templates/cluster_fodc_proxy_deployment.yaml | 20 ++++++++---
 chart/templates/cluster_liaison_statefulset.yaml   | 13 +++++++-
 chart/templates/standalone_statefulset.yaml        | 13 +++++++-
 chart/values.yaml                                  | 10 +++---
 test/e2e/values.fodc.yaml                          | 25 ++++++++++----
 test/e2e/values.lifecycle.yaml                     | 22 ++++++++++++
 7 files changed, 122 insertions(+), 20 deletions(-)

diff --git a/chart/templates/cluster_data_statefulset.yaml 
b/chart/templates/cluster_data_statefulset.yaml
index bd6f698..24283bc 100644
--- a/chart/templates/cluster_data_statefulset.yaml
+++ b/chart/templates/cluster_data_statefulset.yaml
@@ -335,7 +335,18 @@ spec:
               containerPort: {{ $.Values.cluster.fodc.agent.metricsPort }}
           {{- if and $.Values.cluster.fodc.agent.resources (or 
$.Values.cluster.fodc.agent.resources.requests 
$.Values.cluster.fodc.agent.resources.limits) }}
           resources:
-{{ toYaml $.Values.cluster.fodc.agent.resources | indent 12 }}
+            {{- if $.Values.cluster.fodc.agent.resources.requests }}
+            requests:
+              {{- range $request := 
$.Values.cluster.fodc.agent.resources.requests }}
+                {{ $request.key }}: {{ $request.value }}
+              {{- end }}
+            {{- end }}
+            {{- if $.Values.cluster.fodc.agent.resources.limits }}
+            limits:
+              {{- range $limit := $.Values.cluster.fodc.agent.resources.limits 
}}
+                {{ $limit.key }}: {{ $limit.value }}
+              {{- end }}
+            {{- end }}
           {{- end }}
         {{- end }}
         {{- if $roleConfig.backupSidecar.enabled }}
@@ -380,7 +391,18 @@ spec:
           {{- end }}
           {{- with $roleConfig.backupSidecar.resources }}
           resources:
-            {{- toYaml . | nindent 4 }}
+            {{- if $roleConfig.backupSidecar.resources.requests }}
+            requests:
+              {{- range $request := 
$roleConfig.backupSidecar.resources.requests }}
+                {{ $request.key }}: {{ $request.value }}
+                {{- end }}
+            {{- end }}
+            {{- if $roleConfig.backupSidecar.resources.limits }}
+            limits:
+              {{- range $limit := $roleConfig.backupSidecar.resources.limits }}
+                {{ $limit.key }}: {{ $limit.value }}
+                {{- end }}
+            {{- end }}
           {{- end }}
         {{- end }}
         {{- if $roleConfig.lifecycleSidecar.enabled }}
@@ -444,7 +466,18 @@ spec:
           {{- end }}
           {{- with $roleConfig.lifecycleSidecar.resources }}
           resources:
-            {{- toYaml . | nindent 4 }}
+            {{- if $roleConfig.lifecycleSidecar.resources.requests }}
+            requests:
+              {{- range $request := 
$roleConfig.lifecycleSidecar.resources.requests }}
+                {{ $request.key }}: {{ $request.value }}
+                {{- end }}
+            {{- end }}
+            {{- if $roleConfig.lifecycleSidecar.resources.limits }}
+            limits:
+              {{- range $limit := 
$roleConfig.lifecycleSidecar.resources.limits }}
+                {{ $limit.key }}: {{ $limit.value }}
+                {{- end }}
+            {{- end }}
           {{- end }}
         {{- end }}
         {{- range $sidecar := $roleConfig.sidecar }}
diff --git a/chart/templates/cluster_fodc_proxy_deployment.yaml 
b/chart/templates/cluster_fodc_proxy_deployment.yaml
index 428c1f1..e807992 100644
--- a/chart/templates/cluster_fodc_proxy_deployment.yaml
+++ b/chart/templates/cluster_fodc_proxy_deployment.yaml
@@ -104,11 +104,21 @@ spec:
             timeoutSeconds: {{ 
.Values.cluster.fodc.proxy.startupProbe.timeoutSeconds }}
             successThreshold: {{ 
.Values.cluster.fodc.proxy.startupProbe.successThreshold }}
             failureThreshold: {{ 
.Values.cluster.fodc.proxy.startupProbe.failureThreshold }}
-
-      {{- if and .Values.cluster.fodc.proxy.resources (or 
.Values.cluster.fodc.proxy.resources.requests 
.Values.cluster.fodc.proxy.resources.limits) }}
-      resources:
-{{ toYaml .Values.cluster.fodc.proxy.resources | indent 8 }}
-      {{- end }}
+          {{- if and .Values.cluster.fodc.proxy.resources (or 
.Values.cluster.fodc.proxy.resources.requests 
.Values.cluster.fodc.proxy.resources.limits) }}
+          resources:
+            {{- if .Values.cluster.fodc.proxy.resources.requests }}
+            requests:
+              {{- range $request := 
.Values.cluster.fodc.proxy.resources.requests }}
+              {{ $request.key }}: {{ $request.value }}
+              {{- end }}
+            {{- end }}
+            {{- if .Values.cluster.fodc.proxy.resources.limits }}
+            limits:
+              {{- range $limit := .Values.cluster.fodc.proxy.resources.limits 
}}
+              {{ $limit.key }}: {{ $limit.value }}
+              {{- end }}
+            {{- end }}
+          {{- end }}
 
       {{- if .Values.cluster.fodc.proxy.tolerations }}
       tolerations:
diff --git a/chart/templates/cluster_liaison_statefulset.yaml 
b/chart/templates/cluster_liaison_statefulset.yaml
index 3473428..f0f0333 100644
--- a/chart/templates/cluster_liaison_statefulset.yaml
+++ b/chart/templates/cluster_liaison_statefulset.yaml
@@ -325,7 +325,18 @@ spec:
               containerPort: {{ .Values.cluster.fodc.agent.metricsPort }}
           {{- if and .Values.cluster.fodc.agent.resources (or 
.Values.cluster.fodc.agent.resources.requests 
.Values.cluster.fodc.agent.resources.limits) }}
           resources:
-{{ toYaml .Values.cluster.fodc.agent.resources | indent 12 }}
+            {{- if .Values.cluster.fodc.agent.resources.requests }}
+            requests:
+              {{- range $request := 
.Values.cluster.fodc.agent.resources.requests }}
+              {{ $request.key }}: {{ $request.value }}
+              {{- end }}
+            {{- end }}
+            {{- if .Values.cluster.fodc.agent.resources.limits }}
+            limits:
+              {{- range $limit := .Values.cluster.fodc.agent.resources.limits 
}}
+              {{ $limit.key }}: {{ $limit.value }}
+              {{- end }}
+            {{- end }}
           {{- end }}
         {{- end }}
 
diff --git a/chart/templates/standalone_statefulset.yaml 
b/chart/templates/standalone_statefulset.yaml
index eb3f34c..5e2f47e 100644
--- a/chart/templates/standalone_statefulset.yaml
+++ b/chart/templates/standalone_statefulset.yaml
@@ -273,7 +273,18 @@ spec:
               containerPort: {{ .Values.cluster.fodc.agent.metricsPort }}
           {{- if and .Values.cluster.fodc.agent.resources (or 
.Values.cluster.fodc.agent.resources.requests 
.Values.cluster.fodc.agent.resources.limits) }}
           resources:
-{{ toYaml .Values.cluster.fodc.agent.resources | indent 12 }}
+            {{- if .Values.cluster.fodc.agent.resources.requests }}
+            requests:
+              {{- range $request := 
.Values.cluster.fodc.agent.resources.requests }}
+              {{ $request.key }}: {{ $request.value }}
+              {{- end }}
+            {{- end }}
+            {{- if .Values.cluster.fodc.agent.resources.limits }}
+            limits:
+              {{- range $limit := .Values.cluster.fodc.agent.resources.limits 
}}
+              {{ $limit.key }}: {{ $limit.value }}
+              {{- end }}
+            {{- end }}
           {{- end }}
         {{- end }}
 
diff --git a/chart/values.yaml b/chart/values.yaml
index c601d65..9fb5aa8 100644
--- a/chart/values.yaml
+++ b/chart/values.yaml
@@ -868,9 +868,9 @@ cluster:
       ## Resource requests/limits for Proxy
       resources:
         ## @param cluster.fodc.proxy.resources.requests Resource requests for 
Proxy pods
-        requests: {}
+        requests: []
         ## @param cluster.fodc.proxy.resources.limits Resource limits for 
Proxy pods
-        limits: {}
+        limits: []
       image:
         ## @param cluster.fodc.proxy.image.repository Docker repository for 
FODC Proxy
         repository: ghcr.io/apache/skywalking-banyandb-fodc-proxy
@@ -977,10 +977,12 @@ cluster:
       resources:
         ## @param cluster.fodc.agent.resources.requests Resource requests for 
Agent
         requests:
-          memory: 256Mi
+          - key: memory
+            value: 256Mi
         ## @param cluster.fodc.agent.resources.limits Resource limits for Agent
         limits:
-          memory: 256Mi
+          - key: memory
+            value: 256Mi
       ## @param cluster.fodc.agent.metricsPort Metrics port for Agent sidecar 
(prometheus-listen-addr flag)
       metricsPort: 9090
       config:
diff --git a/test/e2e/values.fodc.yaml b/test/e2e/values.fodc.yaml
index 18451e5..a67efd8 100644
--- a/test/e2e/values.fodc.yaml
+++ b/test/e2e/values.fodc.yaml
@@ -254,8 +254,16 @@ cluster:
       podAffinityPreset: ""
       podAntiAffinityPreset: soft
       resources:
-        requests: {}
-        limits: {}
+        requests:
+          - key: cpu
+            value: 200m
+          - key: memory
+            value: 128Mi
+        limits:
+          - key: cpu
+            value: 500m
+          - key: memory
+            value: 512Mi
       image:
         repository: ghcr.io/apache/skywalking-banyandb-fodc-proxy
         tag: "67cb3eaa71d1b2ba9a56dddc17050fd69da78593"
@@ -313,11 +321,16 @@ cluster:
       env: []
       containerSecurityContext: {}
       resources:
-        requests: {
-          memory: 256Mi
-        }
+        requests:
+          - key: cpu
+            value: 200m
+          - key: memory
+            value: 128Mi
         limits:
-          memory: 256Mi
+          - key: cpu
+            value: 500m
+          - key: memory
+            value: 512Mi
       metricsPort: 9090
       config:
         pollMetricsInterval: 15s
diff --git a/test/e2e/values.lifecycle.yaml b/test/e2e/values.lifecycle.yaml
index af24ab0..7d811f3 100644
--- a/test/e2e/values.lifecycle.yaml
+++ b/test/e2e/values.lifecycle.yaml
@@ -193,12 +193,34 @@ cluster:
           enabled: true
           progressFile: "/tmp/stream/lifecycle/progress.json"
           reportDir: "/tmp/stream/lifecycle/reports"
+          resources:
+            requests:
+              - key: cpu
+                value: 200m
+              - key: memory
+                value: 128Mi
+            limits:
+              - key: cpu
+                value: 500m
+              - key: memory
+                value: 512Mi
       warm:
         lifecycleSidecar:
           schedule: "@daily"
           enabled: true
           progressFile: "/tmp/stream/lifecycle/progress.json"
           reportDir: "/tmp/stream/lifecycle/reports"
+          resources:
+            requests:
+              - key: cpu
+                value: 200m
+              - key: memory
+                value: 128Mi
+            limits:
+              - key: cpu
+                value: 500m
+              - key: memory
+                value: 512Mi
       cold:
         replicas: 1
 

Reply via email to