commit c1e95eec5660abd9540400784e9777257d7abcac
author Sebastien Premont-Tendland <sebastien.premont@bell.ca> Tue May 26 18:32:33 2020 -0400
committer Sebastien Premont-Tendland <sebastien.premont@bell.ca> Tue May 26 18:38:58 2020 -0400
tree da1c36a01cb3a54f4debfb817c1b69564c0721f0
parent dbab15fde615d85748f4664bf16376f5643453ac

Fix readiness-check ready.py script for stateful set

Only check the status field readyReplicas instead of
currentReplicas and updatedReplicas. The value of currentReplicas
relates to the number of pods created with the first revision of
the stateful set. If a change is made to the stateful set and pods are restarted
then currentReplicas is no longer returned in the status.
Check JIRA for more information.

Issue-ID: OOM-2418
Signed-off-by: Sebastien Premont-Tendland <sebastien.premont@bell.ca>
Change-Id: If5dbc892b6d807e6ca681457f9874613fce7db04

kubernetes/readiness/src/main/scripts/ready.py

diff --git a/kubernetes/readiness/src/main/scripts/ready.py b/kubernetes/readiness/src/main/scripts/ready.py
index 9393207..b932b04 100644
--- a/kubernetes/readiness/src/main/scripts/ready.py
+++ b/kubernetes/readiness/src/main/scripts/ready.py
@@ -60,10 +60,8 @@
     try:
         response = api.read_namespaced_stateful_set(statefulset_name, namespace)
         s = response.status
-        if (s.updated_replicas == response.spec.replicas and
-                s.replicas == response.spec.replicas and
+        if (s.replicas == response.spec.replicas and
                 s.ready_replicas == response.spec.replicas and
-                s.current_replicas == response.spec.replicas and
                 s.observed_generation == response.metadata.generation):
             log.info("Statefulset " + statefulset_name + "  is ready")
             return True