alvinzhangdd opened a new issue, #12113: URL: https://github.com/apache/pinot/issues/12113
Made this change previously https://github.com/apache/pinot/pull/11800 But it is not backward compatible, if someone uses the following in their helm template: ``` probes: livenessEnabled: true readinessEnabled: true ``` It will throw error: Error: INSTALLATION FAILED: template: pinot-v2/charts/pinot/templates/server/statefulset.yaml:93:26: executing "pinot-v2/charts/pinot/templates/server/statefulset.yaml" at <.Values.server.probes.livenessProbe.endpoint>: nil pointer evaluating interface {}.endpoint It is because the previous commit uses `{{- if .Values.server.probes.readinessProbe.endpoint }}` but never checked the existence of `.Values.server.probes.readinessProbe` The above template setting should fall back to use `.Values.server.probes.endpoint` instead of throwing error. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
