]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
mon/MDSMonitor: remove redundant state change check 47902/head
author胡玮文 <huww98@outlook.com>
Sun, 9 Jan 2022 15:17:38 +0000 (23:17 +0800)
committerVenky Shankar <vshankar@redhat.com>
Thu, 1 Sep 2022 13:39:15 +0000 (19:09 +0530)
commit50b9c4cd09b20d1f9fc012f0146937e86b3647ec
treea81005d58798442296c8d34e786f787774a260f8
parentaf3fe0a0a96ce3f497eb27cb47d795c9f45c4932
mon/MDSMonitor: remove redundant state change check

There are two sets of checks to state change in prepare_beacon.
Since the last commit, many of these checks are covered by
`MDSMap::state_transition_valid`. So merging these checks.

This fixes the bug that standby-replay is evicted unexpectedly.
This bug is introduced in
794d13c9ff4 (mon/MDSMonitor: reject illegal want_states from MDS)
but only reveal itself after
20509bb6c82 (MDSMonitor: handle damaged from standby-replay)

Fixes: https://tracker.ceph.com/issues/53811
Signed-off-by: 胡玮文 <huww98@outlook.com>
(cherry picked from commit bf75a9ea08084afe4a02083473a7146cb91dae3b)
src/mon/MDSMonitor.cc