Skip to content

Commit

Permalink
fix(appset): keep reconcile even when params error occurred
Browse files Browse the repository at this point in the history
Signed-off-by: Or Koren <or.koren@similarweb.com>
  • Loading branch information
Or Koren authored and similark committed Jan 31, 2024
1 parent 28f362b commit a6b4170
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion applicationset/controllers/applicationset_controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,9 @@ func (r *ApplicationSetReconciler) Reconcile(ctx context.Context, req ctrl.Reque
Status: argov1alpha1.ApplicationSetConditionStatusTrue,
}, parametersGenerated,
)
return ctrl.Result{}, err
if len(desiredApplications) < 1 {
return ctrl.Result{}, err
}
}

parametersGenerated = true
Expand Down

0 comments on commit a6b4170

Please sign in to comment.