Publish chart versions that are SemVer 2 compatible

[kengibous]

Describe the current behavior
Currently chart versions are being published with a prefix, "v" (example v2.2.0) which is not SemVer 2 compatible. Such charts like aad-pod-identity are not including "v" in the version.

Describe the improvement
Consider dropping the "v" from publish charts so that the helm manifest provides SemVer 2 compatible versions.

Additional context
I believe the change could be as straightforward as changing - https://github.com/Azure/azure-service-operator/blob/main/v2/charts/azure-service-operator/Chart.yaml#L3 - to version: 2.2.0

[matthchr]

We'll look into this - need to think about impact on existing users, etc as well.

[matthchr]

Looking at other tools:
https://artifacthub.io/packages/helm/cert-manager/cert-manager doesn't have a v prefix for the chart version.
https://artifacthub.io/packages/helm/prometheus-community/prometheus - also no v prefix.

helm docs also suggest we probably shouldn't have a prefix: https://helm.sh/docs/chart_best_practices/conventions/#version-numbers

If we were to make this change, I think we'd make it only going forward (so 2.3.0).

[kengibous]

@matthchr sounds good, I appreciate the consideration.

[kengibous]

@matthchr - i see that version 2.30 is released, but i do not see a published chart

[matthchr]

There was an issue with the 2.3.0 chart (unrelated to this issue). See #3259. We're fixing that and will republish the chart later today.

Sorry about that