Add support for setting kube version on data.helm_template.

[nuttmeister]

Description

When using the helm_template data source on a template that has kube version constraints it will fail with an error (unless v1.20.0) is within that range.

Error: chart requires kubeVersion: >=1.22.0-0 which is incompatible with Kubernetes v1.20.0

So this PR adds the ability to set the kube-version, much like you can with helm template --kube-version.

Usage

data "helm_template" "example" {
  name         = "example"
  namespace    = "default"
  repository   = "https://example.com/repo"
  chart        = "example"
  kube_version = "1.22"
}

Acceptance tests

Be able to template where templates require k8s versions > 1.20.0.

Release Note

Release note for CHANGELOG:

Ability to specify kube-version when using data source template.

References

Fix: #994

Community Note

• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
• If you are interested in working on this issue or have submitted a pull request, please leave a comment

[nuttmeister]

Can see that there already is an open PR for this. Apparently my searching skills today was so-so. #917 .

[BBBmau]

@nuttmeister Hello! Thank you for opening this PR. It all looks great, only thing missing is the documentation for the kube_version attribute that's new to the data_template resource.

Once the documentation is added we should be able to merge it.

[nuttmeister]

@BBBmau thanks. I will add it later today.

[nuttmeister]

@BBBmau hope this is enough docs wise, let me know if more is needed.

[laupow]

@nuttmeister appreciate this PR, looking for that functionality too

[BBBmau]

@BBBmau hope this is enough docs wise, let me know if more is needed.

Thanks for adding the docs! Could you use the description from the helm godoc? Gives a bit of a better description of the kubeversion type

https://github.com/helm/helm/blob/v3.10.2/pkg/action/install.go#L92-L94

[nuttmeister]

@BBBmau does it look better now? The reason why I phrased it the way I did was for it to be in line with the docs/comments on APIVersions specifically. Perhaps it would be good to update the docs for APIVersions as well?

[BBBmau]

@nuttmeister Thanks again for your contribution!

[github-actions]

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.