templating issue in cleanup-webhook.yaml

[al8ba]

The new cleanup-webhook.yaml has an issue when deploying the chart with a .Values.webhook.tolerations defined. The chart uses the tolerations via:

  {{- if .Values.webhook.tolerations }}
  tolerations:
  {{ toYaml .Values.webhook.tolerations | indent 8 }}
  {{- end }}

When rendered, this indents the tolerations incorrectly due to the spaces at the start of the 'toYaml' line (see deployment-admission-server.yaml for how this should be used).

However, that said, does this actually need to pull in tolerations? This is a template for a Job, it does not have a nodeSelector that may need accompanying tolerations - I suspect the four lines above could just be removed.

[kmova]

Fixed by #224