[camel-k] branch master updated: fix: Do not report falsy readiness condition on Integration downscaling

[as...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

astefanutti pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/camel-k.git


The following commit(s) were added to refs/heads/master by this push:
     new 18f0997  fix: Do not report falsy readiness condition on Integration downscaling
18f0997 is described below

commit 18f0997c9a885d06ecaaed1025295dd2fa1bac30
Author: Antonin Stefanutti <an...@stefanutti.fr>
AuthorDate: Tue Mar 9 10:12:31 2021 +0100

    fix: Do not report falsy readiness condition on Integration downscaling
---
 pkg/util/kubernetes/conditions.go | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/pkg/util/kubernetes/conditions.go b/pkg/util/kubernetes/conditions.go
index 88e5465..1c83fca 100644
--- a/pkg/util/kubernetes/conditions.go
+++ b/pkg/util/kubernetes/conditions.go
@@ -76,7 +76,11 @@ func mirrorReadyConditionFromReplicaSet(ctx context.Context, c client.Client, it
 	if rs.Spec.Replicas != nil {
 		replicas = *rs.Spec.Replicas
 	}
-	if replicas == rs.Status.ReadyReplicas {
+	// The Integration is considered ready when the number of replicas
+	// reported to be ready is larger or equal to the specified number
+	// of replicas. This avoid reporting a falsy readiness condition
+	// when the Integration is being down-scaled.
+	if replicas <= rs.Status.ReadyReplicas {
 		it.Status.SetCondition(
 			v1.IntegrationConditionReady,
 			corev1.ConditionTrue,