DandyDeveloper / charts

Various helm charts migrated from [helm/stable] due to deprecation
https://dandydeveloper.github.io/charts
Apache License 2.0
157 stars 145 forks source link

Bugfix priorityClassName #288

Closed maksagit closed 2 months ago

maksagit commented 3 months ago

Bugfix priorityClassName

jkrusy commented 3 months ago

@maksagit just stumbled over the same bug. But your solution won't work. You are missing an end

And

      {{- with .Values.priorityClassName | default .Values.global.priorityClassName }}
      priorityClassName: {{ . }}
      {{- end }}

should do the same and keeping the default

DandyDeveloper commented 2 months ago

Closing this one off in favour of the other.