Skip to content

Commit

Permalink
fix: accommodate the case where readyVGCount is greater than expected…
Browse files Browse the repository at this point in the history
…VGCount

Signed-off-by: Suleyman Akbas <[email protected]>
  • Loading branch information
suleymanakbas91 authored and openshift-cherrypick-robot committed Apr 23, 2024
1 parent cfd0a45 commit 084a2c9
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion internal/controllers/lvmcluster/controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -293,7 +293,7 @@ func (r *Reconciler) updateLVMClusterStatus(ctx context.Context, instance *lvmv1
instance.Status.State = lvmv1alpha1.LVMStatusFailed
} else if isDegraded {
instance.Status.State = lvmv1alpha1.LVMStatusDegraded
} else if isReady && expectedVGCount == readyVGCount {
} else if isReady && expectedVGCount <= readyVGCount { // readyVGCount can be greater than expectedVGCount in case there are taints on some nodes, e.g. during an upgrade. See https://issues.redhat.com/browse/OCPBUGS-30830 for further details.
instance.Status.State = lvmv1alpha1.LVMStatusReady
instance.Status.Ready = true
}
Expand Down

0 comments on commit 084a2c9

Please sign in to comment.