workload controller: delays reporting degraded condition when progres…

…sing > 15 minutes

pkg/operator/apiserver/controller/workload/workload.go

@@ -323,9 +323,11 @@ func (c *Controller) updateOperatorStatus(previousStatus *operatorv1.OperatorSta
 	return nil
 }
 
+// isUpdatingTooLong determines if updating operands takes too long.
+// it returns true if the progressing condition has been set to True for at least 15 minutes
 func isUpdatingTooLong(operatorStatus *operatorv1.OperatorStatus, progressingConditionType string) (bool, error) {
 	progressing := v1helpers.FindOperatorCondition(operatorStatus.Conditions, progressingConditionType)
-	return progressing != nil && progressing.Status == operatorv1.ConditionTrue && time.Now().After(progressing.LastTransitionTime.Add(5*time.Minute)), nil
+	return progressing != nil && progressing.Status == operatorv1.ConditionTrue && time.Now().After(progressing.LastTransitionTime.Add(15*time.Minute)), nil
 }
 
 // EnsureAtMostOnePodPerNode updates the deployment spec to prevent more than

pkg/operator/apiserver/controller/workload/workload_test.go

@@ -140,7 +140,7 @@ func TestUpdateOperatorStatus(t *testing.T) {
 					Type:               fmt.Sprintf("%sDeployment%s", defaultControllerName, operatorv1.OperatorStatusTypeProgressing),
 					Status:             operatorv1.ConditionTrue,
 					Reason:             "PodsUpdating",
-					LastTransitionTime: metav1.NewTime(time.Now().Add(-6 * time.Minute)),
+					LastTransitionTime: metav1.NewTime(time.Now().Add(-16 * time.Minute)),
 				},
 			},
 			validateOperatorStatus: func(actualStatus *operatorv1.OperatorStatus) error {