Skip to content

Commit

Permalink
Add kube_version variable to ensure helm renders manifests correctly
Browse files Browse the repository at this point in the history
We need to set the kube_version helm parameter in order to ensure that
commodore will render the manifests for the correct kubernetes version

Signed-off-by: Nicolas Bigler <[email protected]>
  • Loading branch information
TheBigLee committed Sep 3, 2024
1 parent b597b3a commit 6cd45ad
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 0 deletions.
1 change: 1 addition & 0 deletions class/cert-manager.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
4 changes: 4 additions & 0 deletions class/defaults.yml
Original file line number Diff line number Diff line change
@@ -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
Expand Down

0 comments on commit 6cd45ad

Please sign in to comment.