Skip to content

Commit

Permalink
mon: fix mon scaledown when mons are portable
Browse files Browse the repository at this point in the history
in case of portable mons, mon scaledown was skipped
with below code
```
if mon.NodeName == "" {
			logger.Debugf("mon %q is not scheduled to a specific host", mon.DaemonName)
			continue
		}
```
which skips mon removal if mon nodeName is empty
but if mons are scale down in case of portable mons
in that case also we want to remove extra mons to match
the cephCluster configuration.

Signed-off-by: subhamkrai <srai@redhat.com>
  • Loading branch information
subhamkrai committed Apr 25, 2024
1 parent 44b8499 commit d23a843
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions pkg/operator/ceph/cluster/mon/health.go
Expand Up @@ -450,6 +450,7 @@ func (c *Cluster) determineExtraMonToRemove() string {
for _, mon := range mons {
if mon.NodeName == "" {
logger.Debugf("mon %q is not scheduled to a specific host", mon.DaemonName)
arbitraryMon = mon.DaemonName
continue
}
// Check if there are multiple mons on the node
Expand Down

0 comments on commit d23a843

Please sign in to comment.