[Helm][minor] Make "disabled" flag for worker groups optional (ray-pr…

…oject#548) In this PR, I give the disabled key a default value to make it become an optional field.
kevin85421 · Sep 12, 2022 · 25d0f2b · 25d0f2b
1 parent f8c37fe
commit 25d0f2b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/helm-chart/ray-cluster/templates/raycluster-cluster.yaml b/helm-chart/ray-cluster/templates/raycluster-cluster.yaml
@@ -102,7 +102,7 @@ spec:
 {{ include "ray-cluster.labels" $ | indent 10 }}
   {{- end }}
   {{- end }}
-  {{- if ne .Values.worker.disabled true }}
+  {{- if ne (.Values.worker.disabled | default false) true }}
   - rayStartParams:
     {{- range $key, $val := .Values.worker.initArgs }}
       {{ $key }}: {{ $val | quote }}

diff --git a/helm-chart/ray-cluster/values.yaml b/helm-chart/ray-cluster/values.yaml
@@ -53,8 +53,8 @@ head:
 
 worker:
   # If you want to disable the default workergroup
-  # change the line below
-  disabled: false
+  # uncomment the line below
+  # disabled: true
   groupName: workergroup
   replicas: 1
   type: worker