feat(HelmChartTemplateSpec): optionally ignore missing valuesFiles

Signed-off-by: Robin Breathe <robin@isometry.net>
fluxcd · May 2, 2024 · 2d3bb0c · 2d3bb0c
1 parent b31701e
commit 2d3bb0c
Show file tree

Hide file tree

Showing 6 changed files with 53 additions and 4 deletions.
diff --git a/api/v2beta2/helmrelease_types.go b/api/v2beta2/helmrelease_types.go
@@ -376,6 +376,10 @@ type HelmChartTemplateSpec struct {
 	// +deprecated
 	ValuesFile string `json:"valuesFile,omitempty"`
 
+	// IgnoreMissingValuesFiles controls whether to silently ignore missing values files rather than failing.
+	// +optional
+	IgnoreMissingValuesFiles bool `json:"ignoreMissingValuesFiles,omitempty"`
+
 	// Verify contains the secret name containing the trusted public keys
 	// used to verify the signature and specifies which provider to use to check
 	// whether OCI image is authentic.

diff --git a/config/crd/bases/helm.toolkit.fluxcd.io_helmreleases.yaml b/config/crd/bases/helm.toolkit.fluxcd.io_helmreleases.yaml
@@ -1320,6 +1320,10 @@ spec:
                         maxLength: 2048
                         minLength: 1
                         type: string
+                      ignoreMissingValuesFiles:
+                        description: IgnoreMissingValuesFiles controls whether to
+                          silently ignore missing values files rather than failing.
+                        type: boolean
                       interval:
                         description: |-
                           Interval at which to check the v1.Source for updates. Defaults to

diff --git a/docs/api/v2beta1/helm.md b/docs/api/v2beta1/helm.md
@@ -604,6 +604,18 @@ ValuesFiles items. Ignored when omitted.</p>
 </tr>
 <tr>
 <td>
+<code>ignoreMissingValuesFiles</code><br>
+<em>
+bool
+</em>
+</td>
+<td>
+<em>(Optional)</em>
+<p>Whether to silently ignore missing values files rather than failing.</p>
+</td>
+</tr>
+<tr>
+<td>
 <code>verify</code><br>
 <em>
 <a href="#helm.toolkit.fluxcd.io/v2beta1.HelmChartTemplateVerification">

diff --git a/docs/api/v2beta2/helm.md b/docs/api/v2beta2/helm.md
@@ -807,6 +807,18 @@ ValuesFiles items. Ignored when omitted.</p>
 </tr>
 <tr>
 <td>
+<code>ignoreMissingValuesFiles</code><br>
+<em>
+bool
+</em>
+</td>
+<td>
+<em>(Optional)</em>
+<p>IgnoreMissingValuesFiles controls whether to silently ignore missing values files rather than failing.</p>
+</td>
+</tr>
+<tr>
+<td>
 <code>verify</code><br>
 <em>
 <a href="#helm.toolkit.fluxcd.io/v2beta2.HelmChartTemplateVerification">
@@ -999,6 +1011,18 @@ ValuesFiles items. Ignored when omitted.</p>
 </tr>
 <tr>
 <td>
+<code>ignoreMissingValuesFiles</code><br>
+<em>
+bool
+</em>
+</td>
+<td>
+<em>(Optional)</em>
+<p>IgnoreMissingValuesFiles controls whether to silently ignore missing values files rather than failing.</p>
+</td>
+</tr>
+<tr>
+<td>
 <code>verify</code><br>
 <em>
 <a href="#helm.toolkit.fluxcd.io/v2beta2.HelmChartTemplateVerification">

diff --git a/docs/spec/v2beta1/helmreleases.md b/docs/spec/v2beta1/helmreleases.md
@@ -215,6 +215,10 @@ type HelmChartTemplateSpec struct {
 	// +optional
 	// +deprecated
 	ValuesFile string `json:"valuesFile,omitempty"`
+
+	// IgnoreMissingValuesFiles controls whether to silently ignore missing values files rather than failing.
+	// +optional
+	IgnoreMissingValuesFiles bool `json:"ignoreMissingValuesFiles,omitempty"`
 }
 
 // Install holds the configuration for Helm install actions performed for this

diff --git a/internal/reconcile/helmchart_template.go b/internal/reconcile/helmchart_template.go
@@ -226,10 +226,11 @@ func buildHelmChartFromTemplate(obj *v2.HelmRelease) *sourcev1.HelmChart {
 				Name: template.Spec.SourceRef.Name,
 				Kind: template.Spec.SourceRef.Kind,
 			},
-			Interval:          template.GetInterval(obj.Spec.Interval),
-			ReconcileStrategy: template.Spec.ReconcileStrategy,
-			ValuesFiles:       template.Spec.ValuesFiles,
-			ValuesFile:        template.Spec.ValuesFile,
+			Interval:                 template.GetInterval(obj.Spec.Interval),
+			ReconcileStrategy:        template.Spec.ReconcileStrategy,
+			ValuesFiles:              template.Spec.ValuesFiles,
+			ValuesFile:               template.Spec.ValuesFile,
+			IgnoreMissingValuesFiles: template.Spec.IgnoreMissingValuesFiles,
 		},
 	}
 	if verifyTpl := template.Spec.Verify; verifyTpl != nil {