Merge pull request #157 from projectsyn/configurable_k8s_version

Add kube_version variable to ensure helm renders manifests correctly

class/cert-manager.yml

@@ -20,6 +20,7 @@ parameters:
         helm_params:
           name: cert-manager
           namespace: ${cert_manager:namespace}
+          kube_version: ${cert_manager:kubernetes_version}
       - output_path: cert-manager/
         input_type: jsonnet
         output_type: yaml

class/defaults.yml

@@ -1,6 +1,10 @@
 parameters:
   cert_manager:
     namespace: syn-cert-manager
+
+    # Used when rendering the Helm chart
+    kubernetes_version: "1.27"
+
     dns01-recursive-nameservers: "1.1.1.1:53"
     charts:
       cert-manager: v1.13.5

docs/modules/ROOT/pages/references/parameters.adoc

@@ -13,6 +13,16 @@ The namespace in which to install cert-manager.
 The component always adds label `openshift.io/cluster-monitoring=true` to the namespace.
 Additionally, if component `prometheus` is installed on the cluster, the component registers the namespace to be monitored through the default Prometheus stack managed by that component.
 
+== `kubernetes_version`
+
+[horizontal]
+type:: string
+default:: https://github.com/projectsyn/component-cert-manager/blob/master/class/defaults.yml[See `class/defaults.yml`]
+
+The Kubernetes version to provide to `helm template` when rendering the Helm chart.
+
+TIP: Set this parameter to `${dynamic_facts:kubernetesVersion:major}.${dynamic_facts:kubernetesVersion.minor}` to use the cluster's reported Kubernetes version when rendering the Helm chart.
+
 == `charts:cert-manager`
 
 [horizontal]