break before default
Signed-off-by: Raffael Sahli <raffael.sahli@doodle.com>
This commit is contained in:
parent
bc3c4e2a36
commit
1e19503359
|
@ -403,13 +403,8 @@ func (r *HelmChartReconciler) reconcileFromHelmRepository(ctx context.Context,
|
||||||
if changed, err := helm.OverwriteChartDefaultValues(helmChart, valuesData); err != nil {
|
if changed, err := helm.OverwriteChartDefaultValues(helmChart, valuesData); err != nil {
|
||||||
return sourcev1.HelmChartNotReady(chart, sourcev1.ChartPackageFailedReason, err.Error()), err
|
return sourcev1.HelmChartNotReady(chart, sourcev1.ChartPackageFailedReason, err.Error()), err
|
||||||
} else if !changed {
|
} else if !changed {
|
||||||
// No changes, write original package to storage
|
// No changes, skip to write original package to storage
|
||||||
if err := r.Storage.AtomicWriteFile(&newArtifact, res, 0644); err != nil {
|
goto skipToDefault
|
||||||
err = fmt.Errorf("unable to write chart file: %w", err)
|
|
||||||
return sourcev1.HelmChartNotReady(chart, sourcev1.StorageOperationFailedReason, err.Error()), err
|
|
||||||
}
|
|
||||||
|
|
||||||
break
|
|
||||||
}
|
}
|
||||||
|
|
||||||
// Create temporary working directory
|
// Create temporary working directory
|
||||||
|
@ -435,6 +430,9 @@ func (r *HelmChartReconciler) reconcileFromHelmRepository(ctx context.Context,
|
||||||
|
|
||||||
readyMessage = fmt.Sprintf("Fetched and packaged revision: %s", newArtifact.Revision)
|
readyMessage = fmt.Sprintf("Fetched and packaged revision: %s", newArtifact.Revision)
|
||||||
readyReason = sourcev1.ChartPackageSucceededReason
|
readyReason = sourcev1.ChartPackageSucceededReason
|
||||||
|
break
|
||||||
|
skipToDefault:
|
||||||
|
fallthrough
|
||||||
default:
|
default:
|
||||||
// Write artifact to storage
|
// Write artifact to storage
|
||||||
if err := r.Storage.AtomicWriteFile(&newArtifact, res, 0644); err != nil {
|
if err := r.Storage.AtomicWriteFile(&newArtifact, res, 0644); err != nil {
|
||||||
|
|
Loading…
Reference in New Issue