]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commit
mon/MDSMonitor: batch last_metadata update with pending 50862/head
authorPatrick Donnelly <pdonnell@redhat.com>
Tue, 4 Apr 2023 18:03:49 +0000 (14:03 -0400)
committerPatrick Donnelly <pdonnell@redhat.com>
Wed, 5 Apr 2023 02:14:41 +0000 (22:14 -0400)
commit6f69fe9739a23974a46a4e13e4c29b431d95acc4
tree498c66445691731bba319195d0c47e703e4f8bec
parent3faafba6e1bc223be071a194fdb10d6c6981c814
mon/MDSMonitor: batch last_metadata update with pending

I believe the problem here is that the last_metadata change is lost in a
ECANCELED/EAGAIN transaction but the pending map change goes through in
the next one. I've been unable to find an exact way to reproduce this.
The problem seems to occur when upgrades are performed which would
indicate shuffling in the monitors where quorum would be lost repeatedly.

This seems to be the most likely explanation so let's go ahead and make
this change even without the reproducer. In any case, it has the added
benefit of batching the pending map update (to up:standby) with the
last_metadata update.

Fixes: https://tracker.ceph.com/issues/24403
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
src/mon/MDSMonitor.cc