review comment

Author: dgrove-oss

internal/controller/appwrapper_controller.go

@@ -181,12 +181,12 @@ func (r *AppWrapperReconciler) Reconcile(ctx context.Context, req ctrl.Request)
 if err != nil {
 return ctrl.Result{}, err
 }
-if podStatus.succeeded >= podStatus.expected && (podStatus.pending+podStatus.running == 0) {
+if podStatus.succeeded >= podStatus.expected && (podStatus.pending+podStatus.running+podStatus.failed == 0) {
 meta.SetStatusCondition(&aw.Status.Conditions, metav1.Condition{
 Type: string(workloadv1beta2.QuotaReserved),
 Status: metav1.ConditionFalse,
 Reason: string(workloadv1beta2.AppWrapperSucceeded),
-Message: fmt.Sprintf("%v pods succeeded and no running or pending pods", podStatus.succeeded),
+Message: fmt.Sprintf("%v pods succeeded and no running, pending, or failed pods", podStatus.succeeded),
 })
 return r.updateStatus(ctx, aw, workloadv1beta2.AppWrapperSucceeded)
 }