]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commit
msg: add alternate statuses for ms_dispatch2 handling
authorPatrick Donnelly <pdonnell@ibm.com>
Tue, 18 Feb 2025 17:41:52 +0000 (12:41 -0500)
committerPatrick Donnelly <pdonnell@ibm.com>
Fri, 28 Feb 2025 00:55:47 +0000 (19:55 -0500)
commitc9d0913f53b0c4d632746075c05c2f2187d77c02
tree0b92e227559140d162d24c84e790c21b6811e02e
parent3b79bdbee04c270c227be3dd1cb6dfe392646f8d
msg: add alternate statuses for ms_dispatch2 handling

Many dispatchers return false to allow other dispatchers also common messages
like MOSDMap or MFSMap. They implicitly depend on some dispatcher which is
always at the "tail" of the dispatcher queue to return "true" indicating the
msg was processed to avoid messages like:

    2025-02-18T05:31:17.738+0000 7f5206546640  0 ms_deliver_dispatch: unhandled message 0x5632d05f0700 fsmap(e 9) from mon.0 v2:172.21.3.230:40412/0

but this cannot always happen when some libraries like the RadosClient used standalone.

So, add a variant for encapsulating other indications for how the message was
processed by dispatch2.  For example, a message may be "acknowledged" but
explicitly allow other dispatchers to try processing the message.

Note: we're using a variant to avoid updating all of the ms_dispatch code to
use the sentinel classes.

Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
22 files changed:
src/client/Client.cc
src/client/Client.h
src/mds/Beacon.cc
src/mds/Beacon.h
src/mds/MDSDaemon.cc
src/mds/MDSDaemon.h
src/mds/MetricAggregator.cc
src/mds/MetricAggregator.h
src/mds/MetricsHandler.cc
src/mds/MetricsHandler.h
src/mgr/DaemonServer.cc
src/mgr/DaemonServer.h
src/mgr/MgrClient.cc
src/mgr/MgrClient.h
src/mgr/MgrStandby.cc
src/mgr/MgrStandby.h
src/msg/Dispatcher.h
src/msg/Messenger.h
src/nvmeof/NVMeofGwMonitorClient.cc
src/nvmeof/NVMeofGwMonitorClient.h
src/tools/cephfs_mirror/ClusterWatcher.cc
src/tools/cephfs_mirror/ClusterWatcher.h