Pod labels updated to include app.kubernetes.io/version

[jw-maynard]

Issue

Description of changes

I would like to see the app version added to pods as a label. In addition I moved the merging of selectorLabels into the podLabels out of the deployment and into a new helper.

I'd be curious if merging additionalLabels into the podLabels would also be a good idea since the description of additionalLabels says they are added to all components but that doesn't include the pods created by the deployment. Perhaps that is intentional.

Also, I copied the code for app.kubernetes.io/managed-by which will default to Helm but perhaps that should be removed since technically the pod is managed by the deployment which is managed by Helm...

Checklist

• Added/modified documentation as required (such as the README.md for modified charts)
• Incremented the chart version in Chart.yaml for the modified chart(s)
• Manually tested. Describe what testing was done in the testing section below
• Make sure the title of the PR is a good description that can go into the release notes

Testing

None, the test scripts seem to only be helpers. It's a pretty small change though.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[jw-maynard]

@kishorj @M00nF1sh are there any changes I need to make here?

[jw-maynard]

Anything that needs to be done here to get this checked and merged?

[jw-maynard]

@kishorj @M00nF1sh any thoughts on this?