Added template expansion for helm chart version (#5709)

[kadern0]

Signed-off-by: Pablo Caderno [email protected]

Fixes: #5709
Related: Relevant tracking issues, for context
Merge before/after: Dependent or prerequisite PRs

Description

Added template expansion for Helm chart version.

Before (no variable expansion):

After (it uses the value):

If variable is not defined:

[codecov]

Codecov Report

Merging #6157 (4774833) into master (62a01f6) will decrease coverage by 0.02%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##           master    #6157      +/-   ##
==========================================
- Coverage   71.12%   71.09%   -0.03%     
==========================================
  Files         483      483              
  Lines       21549    21553       +4     
==========================================
- Hits        15326    15323       -3     
- Misses       5246     5249       +3     
- Partials      977      981       +4     

Impacted Files	Coverage Δ
pkg/skaffold/deploy/helm/deploy.go	76.87% <66.66%> (-0.38%)	⬇️
pkg/skaffold/deploy/helm/args.go	83.19% <100.00%> (ø)
pkg/skaffold/util/tar.go	52.87% <0.00%> (-4.60%)	⬇️
pkg/skaffold/docker/image.go	80.08% <0.00%> (-1.22%)	⬇️
pkg/skaffold/docker/parse.go	88.23% <0.00%> (+0.84%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 62a01f6...4774833. Read the comment docs.

[briandealwis]

Thanks @kadern0! I'll make the change I suggested below and added a quick test for verification.

[briandealwis]

@googlebot I consent.

[kadern0]

Thanks @kadern0! I'll make the change I suggested below and added a quick test for verification.

Thanks :)