jscheffl commented on code in PR #60751:
URL: https://github.com/apache/airflow/pull/60751#discussion_r2702593603
##########
chart/files/pod-template-file.kubernetes-helm-yaml:
##########
@@ -42,7 +42,7 @@ metadata:
{{- end }}
annotations:
{{- toYaml $podAnnotations | nindent 4 }}
- {{- if .Values.workers.kerberosInitContainer.enabled }}
+ {{- if or .Values.workers.kubernetes.kerberosInitContainer.enabled
.Values.workers.kerberosInitContainer.enabled }}
Review Comment:
Same for me actually in regards of inconsistencies. But Semver should be
kept and such we have a couple of deprecations to carry.
Otherwise not all technical combinations make sense, so it is fine (and also
pragmatic) to restrict complexity to a manageable and meaningful scope.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
