Skip to content

Commit 83c8058

Browse files
committed
review comment
1 parent fe2d85e commit 83c8058

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

internal/controller/appwrapper_controller.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -181,7 +181,7 @@ func (r *AppWrapperReconciler) Reconcile(ctx context.Context, req ctrl.Request)
181181
if err != nil {
182182
return ctrl.Result{}, err
183183
}
184-
if podStatus.succeeded >= podStatus.expected && (podStatus.pending+podStatus.running == 0) {
184+
if podStatus.succeeded >= podStatus.expected && (podStatus.pending+podStatus.running+podStatus.failed == 0) {
185185
meta.SetStatusCondition(&aw.Status.Conditions, metav1.Condition{
186186
Type: string(workloadv1beta2.QuotaReserved),
187187
Status: metav1.ConditionFalse,

0 commit comments

Comments
 (0)