From d335dc59eb7c1f6aa60ea7337b0f5e7ac360b656 Mon Sep 17 00:00:00 2001 From: FengJiankui Date: Wed, 27 Apr 2022 14:32:19 +0800 Subject: [PATCH] mon: quorumStatus.MonMap.Mons changed to quorumStatus.Quorum The path use quorum instead of monMap to determine if there is enough mon. (bug #10165). Signed-off-by: FengJiankui --- pkg/operator/ceph/cluster/mon/health.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/operator/ceph/cluster/mon/health.go b/pkg/operator/ceph/cluster/mon/health.go index da53443f6..e645dd423 100644 --- a/pkg/operator/ceph/cluster/mon/health.go +++ b/pkg/operator/ceph/cluster/mon/health.go @@ -161,7 +161,7 @@ func (c *Cluster) checkHealth() error { } else { // when the mon isn't in the clusterInfo, but is in quorum and there are // enough mons, remove it else remove it on the next run - if inQuorum && len(quorumStatus.MonMap.Mons) > desiredMonCount { + if inQuorum && len(quorumStatus.Quorum) > desiredMonCount { logger.Warningf("mon %q not in source of truth but in quorum, removing", mon.Name) if err := c.removeMon(mon.Name); err != nil { logger.Warningf("failed to remove mon %q. %v", mon.Name, err)