Skip to content

Commit cc4302a

Browse files
jiangzhoPeter Toth
authored andcommitted
[SPARK-53186] Fix probe port override from helm chart
### What changes were proposed in this pull request? This is a bug fix to apply the health probe port from helm chart value override onto operator config ### Why are the changes needed? Without this patch, the port override - if set in value.yaml - is only applied to operator container spec but not applied in config, would cause probe failures thereafter ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? CIs, helm lint, and local testing ### Was this patch authored or co-authored using generative AI tooling? No Closes apache#296 from jiangzho/probe_port_override. Authored-by: Zhou JIANG <[email protected]> Signed-off-by: Peter Toth <[email protected]>
1 parent cdd2790 commit cc4302a

File tree

1 file changed

+1
-0
lines changed
  • build-tools/helm/spark-kubernetes-operator/templates

1 file changed

+1
-0
lines changed

build-tools/helm/spark-kubernetes-operator/templates/_helpers.tpl

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -115,6 +115,7 @@ spark.kubernetes.operator.namespace={{ .Release.Namespace }}
115115
spark.kubernetes.operator.name={{- include "spark-operator.name" . }}
116116
spark.kubernetes.operator.dynamicConfig.enabled={{ .Values.operatorConfiguration.dynamicConfig.enable }}
117117
spark.kubernetes.operator.metrics.port={{ include "spark-operator.metricsPort" . }}
118+
spark.kubernetes.operator.health.probePort={{ include "spark-operator.probePort" . }}
118119
{{- if .Values.workloadResources.namespaces.overrideWatchedNamespaces }}
119120
spark.kubernetes.operator.watchedNamespaces={{ include "spark-operator.workloadNamespacesStr" . | trim }}
120121
{{- end }}

0 commit comments

Comments
 (0)