Skip to content

Commit 2d1d959

Browse files
dashanjiarttor
authored andcommitted
Use the latest version of cert-manager
Signed-off-by: dashanji <[email protected]>
1 parent 9f44526 commit 2d1d959

File tree

2 files changed

+9
-3
lines changed

2 files changed

+9
-3
lines changed

pkg/helm/chart.go

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -103,8 +103,15 @@ func overwriteTemplateFile(filename, chartDir string, crd bool, templates []helm
103103

104104
func overwriteValuesFile(chartDir string, values helmify.Values, certManagerAsSubchart bool) error {
105105
if certManagerAsSubchart {
106-
values.Add(true, "cert-manager", "installCRDs")
107-
values.Add(true, "cert-manager", "enabled")
106+
_, err := values.Add(true, "cert-manager", "installCRDs")
107+
if err != nil {
108+
return errors.Wrap(err, "unable to add cert-manager.installCRDs")
109+
}
110+
111+
_, err = values.Add(true, "cert-manager", "enabled")
112+
if err != nil {
113+
return errors.Wrap(err, "unable to add cert-manager.enabled")
114+
}
108115
}
109116
res, err := yaml.Marshal(values)
110117
if err != nil {

pkg/helm/init.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -186,7 +186,6 @@ func chartYAML(appName string, certManagerAsSubchart bool) []byte {
186186
annotatins := `
187187
dependencies:
188188
- name: cert-manager
189-
version: 1.9.1
190189
repository: https://charts.jetstack.io
191190
condition: certManager.enabled`
192191
if certManagerAsSubchart {

0 commit comments

Comments
 (0)