[fluent-bit] optional verbosePodLabels to include app.kubernetes.io/version applied to all pods

[jhulick]

What this PR does / why we need it:

• It adds common labels to metadata labels in the pod spec. The main reasoning is to persist the app.kubernetes.io/version and app.kubernetes.io/name labels to the pod for standardization. Per docs, (https://kubernetes.io/docs/concepts/overview/working-with-objects/common-labels/#labels) "In order to take full advantage of using these labels, they should be applied on every resource object.".

• Improves traceability throughout the application. Ex. via Istio/Kiali dashboard.

[stevehipwell]

Thanks for the PR @jhulick but as was the case for Metrics Server this isn't a desirable change for the chart. If you make this change optional and off by default I think it would be OK; my preference would be to use a value like verbosePodLabels: false.

[jhulick]

@stevehipwell, I was hesitant to create this PR for that reason, but since fluent-bit is data plane rather than control plane I thought it would be considered acceptable in this case, unless there are other reasons? Is it primarily due to telemetry latency? But I'll update per your proposed suggested.

[stevehipwell]

@jhulick I don't think the labels are a good idea, but even if they were we'd still be looking to follow the "principal of least astonishment"; so making them opt in is the only real way of introducing this change.