Fix errors in service account template.

[v-shenoy]

Signed-off-by: Vighnesh Shenoy [email protected]

Provide a description of what has been changed

Checklist

• Commits are signed with Developer Certificate of Origin (DCO - learn more)
• A PR is opened to update KEDA core (repo) (if applicable, ie. when deployment manifests are modified)
• README is updated with new configuration values (if applicable)

Relates to kedacore/keda#2487
Relates to #263

[v-shenoy]

@tomkerkhove Kinda new to helm charts, so I didn't realize I could install from local directories. So I didn't test my changes until I pushed them to the PR. After trying to install from local, I realized I made some mistakes, please merge otherwise the helm changes will throw errors.

Sorry for the inconvenience.

[tomkerkhove]

No problem at all! Normally the CI should have caught that but we don't test all configuration scenarios, unfortunately.

[tomkerkhove]

Let's not merge this PR just yet.

[tomkerkhove]

The convention is {{ .Values.podIdentity.azureWorkload.tokenExpiration | quote }} over {{ quote .Values.podIdentity.azureWorkload.tokenExpiration }} but that's OK.

[v-shenoy]

The convention is {{ .Values.podIdentity.azureWorkload.tokenExpiration | quote }} over {{ quote .Values.podIdentity.azureWorkload.tokenExpiration }} but that's OK.

Oh. I just had a quick glance and I thought | quote would mostly be used if I had multiple functions being chained. My bad.