fix: scaledjobs stuck as not ready (kedacore#6329)

since code was missing for setting a scaledjob as ready it was stuck as unready if there ever was a problem

This is a fix for a regression in kedacore#5916

Signed-off-by: Mårten Svantesson <[email protected]>
Signed-off-by: Jorge Turrado <[email protected]>

Author: msvticket

pkg/scaling/executor/scale_jobs.go

@@ -65,17 +65,26 @@ func (e *scaleExecutor) RequestJobScale(ctx context.Context, scaledJob *kedav1al
 		logger.V(1).Info("No change in activity")
 	}
 
+	readyCondition := scaledJob.Status.Conditions.GetReadyCondition()
 	if isError {
 		// some triggers responded with error
 		// Set ScaledJob.Status.ReadyCondition to Unknown
-		readyCondition := scaledJob.Status.Conditions.GetReadyCondition()
 		msg := "Some triggers defined in ScaledJob are not working correctly"
 		logger.V(1).Info(msg)
 		if !readyCondition.IsUnknown() {
 			if err := e.setReadyCondition(ctx, logger, scaledJob, metav1.ConditionUnknown, "PartialTriggerError", msg); err != nil {
 				logger.Error(err, "error setting ready condition")
 			}
 		}
+	} else if !readyCondition.IsTrue() {
+		// if the ScaledObject's triggers aren't in the error state,
+		// but ScaledJob.Status.ReadyCondition is set not set to 'true' -> set it back to 'true'
+		msg := "ScaledJob is defined correctly and is ready for scaling"
+		logger.V(1).Info(msg)
+		if err := e.setReadyCondition(ctx, logger, scaledJob, metav1.ConditionTrue,
+			"ScaledJobReady", msg); err != nil {
+			logger.Error(err, "error setting ready condition")
+		}
 	}
 
 	condition := scaledJob.Status.Conditions.GetActiveCondition()