Support multiple ImagePullSecrets

[chunter0]

Add support for multiple image pull secrets

Manual Helm Testing

cat <<-EOF | helm template codegen/test/chart/image-pull-secrets -f - | yq 'select(.kind == "Deployment") | .spec.template.spec.imagePullSecrets'
painter:
  enabled: true
  image:
    pullSecret: a-registry
EOF

cat <<-EOF | helm template codegen/test/chart/image-pull-secrets -f - | yq 'select(.kind == "Deployment") | .spec.template.spec.imagePullSecrets'
painter:
  enabled: true
  imagePullSecrets:
  - name: b-registry
EOF

cat <<-EOF | helm template codegen/test/chart/image-pull-secrets -f - | yq 'select(.kind == "Deployment") | .spec.template.spec.imagePullSecrets'
painter:
  enabled: true
  image:
    pullSecret: a-registry
  imagePullSecrets:
  - name: b-registry
EOF

BOT NOTES:
resolves #574

[solo-changelog-bot]

Issues linked to changelog:
#574

[josh-pritchard]

Can we open a draft against GME where you generate the chart against this locally? Then you could verify the templating is correct there.

[josh-pritchard]

I don't think this is going to render right, imagePullSecrets is an array not a dict. Maybe something like this?

Suggested change

	{{- (dict "imagePullSecrets" $pullSecrets) | toYaml | nindent 6 }}
	{{- if gt (len $pullSecrets) 0 -}}
	imagePullSecrets:
	{{- toYaml $pullSecrets | nindent 8 }}
	{{- end}}

[josh-pritchard]

not sure if the indent is right