Use the latest version of cert-manager

Signed-off-by: dashanji <dashanjic@gmail.com>
arttor · Mar 29, 2023 · 2d1d959 · 2d1d959
1 parent 9f44526
commit 2d1d959
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/pkg/helm/chart.go b/pkg/helm/chart.go
@@ -103,8 +103,15 @@ func overwriteTemplateFile(filename, chartDir string, crd bool, templates []helm
 
 func overwriteValuesFile(chartDir string, values helmify.Values, certManagerAsSubchart bool) error {
 	if certManagerAsSubchart {
-		values.Add(true, "cert-manager", "installCRDs")
-		values.Add(true, "cert-manager", "enabled")
+		_, err := values.Add(true, "cert-manager", "installCRDs")
+		if err != nil {
+			return errors.Wrap(err, "unable to add cert-manager.installCRDs")
+		}
+
+		_, err = values.Add(true, "cert-manager", "enabled")
+		if err != nil {
+			return errors.Wrap(err, "unable to add cert-manager.enabled")
+		}
 	}
 	res, err := yaml.Marshal(values)
 	if err != nil {

diff --git a/pkg/helm/init.go b/pkg/helm/init.go
@@ -186,7 +186,6 @@ func chartYAML(appName string, certManagerAsSubchart bool) []byte {
 	annotatins := `
 dependencies:
   - name: cert-manager
-    version: 1.9.1
     repository: https://charts.jetstack.io
     condition: certManager.enabled`
 	if certManagerAsSubchart {