Skip to content

Commit

Permalink
Helm: add service type configuration to head group for ray-cluster (r…
Browse files Browse the repository at this point in the history
…ay-project#614)

Co-authored-by: Iskandar Sitdikov <[email protected]>

Currently service type is not used in Helm and ClusterIP is hardcoded.
This commit fixes that so that the service type in values.yaml is used.
  • Loading branch information
IceKhan13 authored Oct 11, 2022
1 parent 61d887e commit e20c68e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion helm-chart/ray-cluster/templates/raycluster-cluster.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ spec:
enableInTreeAutoscaling: {{ .Values.head.enableInTreeAutoscaling }}
{{- end }}
headGroupSpec:
serviceType: ClusterIP
serviceType: {{ .Values.service.type }}
rayStartParams:
{{- range $key, $val := .Values.head.initArgs }}
{{ $key }}: {{ $val | quote }}
Expand Down

0 comments on commit e20c68e

Please sign in to comment.