This is an automated email from the ASF dual-hosted git repository.
ethanfeng pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/celeborn.git
The following commit(s) were added to refs/heads/main by this push:
new 893a7449e [CELEBORN-1830] Chart statefulset resources key duplicate
893a7449e is described below
commit 893a7449e0d6413a9126f3c79bbc01ae78d75899
Author: pengqli <[email protected]>
AuthorDate: Fri Jan 10 14:30:55 2025 +0800
[CELEBORN-1830] Chart statefulset resources key duplicate
### What changes were proposed in this pull request?
update chart, remove resources key duplicate
### Why are the changes needed?
<img width="622" alt="Screenshot 2025-01-10 at 11 14 47"
src="https://github.com/user-attachments/assets/03d7a13b-0a4b-4708-9b12-873625a9825f";
/>
### Does this PR introduce _any_ user-facing change?
No any user-facing change
### How was this patch tested?
helm template --debug ../celeborn
Closes #3061 from dev-lpq/fix_chart_resources.
Authored-by: pengqli <[email protected]>
Signed-off-by: mingji <[email protected]>
---
charts/celeborn/templates/master/statefulset.yaml | 2 --
charts/celeborn/templates/worker/statefulset.yaml | 2 --
2 files changed, 4 deletions(-)
diff --git a/charts/celeborn/templates/master/statefulset.yaml
b/charts/celeborn/templates/master/statefulset.yaml
index 7d3fe6ecb..f3f3e651a 100644
--- a/charts/celeborn/templates/master/statefulset.yaml
+++ b/charts/celeborn/templates/master/statefulset.yaml
@@ -82,8 +82,6 @@ spec:
echo "waiting for master";
sleep 2;
done && exec /opt/celeborn/sbin/start-master.sh
- resources:
- {{- toYaml .Values.resources.master | nindent 10 }}
ports:
- containerPort: {{ .Values.service.port }}
- containerPort: {{ get .Values.celeborn "celeborn.master.http.port" |
default 9098 }}
diff --git a/charts/celeborn/templates/worker/statefulset.yaml
b/charts/celeborn/templates/worker/statefulset.yaml
index f8d102312..bdcb65783 100644
--- a/charts/celeborn/templates/worker/statefulset.yaml
+++ b/charts/celeborn/templates/worker/statefulset.yaml
@@ -86,8 +86,6 @@ spec:
echo "waiting for master";
sleep 2;
done && exec /opt/celeborn/sbin/start-worker.sh
- resources:
- {{- toYaml .Values.resources.worker | nindent 10 }}
ports:
- containerPort: {{ get .Values.celeborn "celeborn.worker.http.port" |
default 9096 }}
name: metrics
