Add a --kube-version option to pass to helm template

[deriamis]

Currently, templates are only generated with the current Kubernetes version, but it is possible to validate against any schema. This means that it's not actually possible to generate what you intend to validate. This commit adds --kube-version option, which gets passed down to helm template. The option default is whatever --kubernetes-version is set to, if set.

This option is deliberately separate from --kubernetes-version because a) they are passed to two different commands and b) there may be reasons why a user might want to validate against a schema that doesn't match what the template was generated for, such as to validate cross-version compatibility.

[jtyr]

Thanks for creating this PR, @deriamis. Please could you make the default value of kube-version the same like the value of kubernetes-version? Like this the two will be the same by default but if people want to change it, they still can.

[deriamis]

Thanks for creating this PR, @deriamis. Please could you make the default value of kube-version the same like the value of kubernetes-version? Like this the two will be the same by default but if people want to change it, they still can.

Sure, no problem. One moment and I'll push that change - once my keyboard stops being silly.

[deriamis]

@jtyr I've made the change as requested. FYI, I have tested the change already in a Docker env as well:

bash-5.1# helm kubeconform --strict --summary -f ./template_settings.yaml --kubernetes-version "1.19.16" --schema-location default --schema-location 'https://raw.githubusercontent.com/kubernetes/kubernetes/master/api/openapi-spec/v3/apis__apiextensions.k8s.io__v1_openapi.json' --output json .
{
  "resources": [],
  "summary": {
    "valid": 150,
    "invalid": 0,
    "errors": 0,
    "skipped": 0
  }
}

[jtyr]

It looks good. Please bump the version and we can merge it.

[deriamis]

It looks good. Please bump the version and we can merge it.

@jtyr Done!

[jtyr]

Thanks for your contribution, @deriamis.