Skip to content

PTX-3561 Torpedo reports 2/3 pods are ready, when all pods are ready#261

Open
devops-thiago wants to merge 1 commit intoportworx:masterfrom
devops-thiago:PTX-3561
Open

PTX-3561 Torpedo reports 2/3 pods are ready, when all pods are ready#261
devops-thiago wants to merge 1 commit intoportworx:masterfrom
devops-thiago:PTX-3561

Conversation

@devops-thiago
Copy link
Contributor

Signed-off-by: Thiago Gonzaga [email protected]

What this PR does / why we need it:
This PR fixes an issue when kubelet is down in one node, pod is ready and running but ReadyReplicas field has a wrong value


if readyCount >= requiredReplicas {
return "", false, nil
if requiredReplicas > readyCount {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shouldn't this be >=

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants