From: Greg Farnum <gfarnum@redhat.com>
Date: Fri, 16 Feb 2018 23:44:22 +0000 (-0800)
Subject: mon: mark manager beacons as no_reply
X-Git-Tag: wip-pdonnell-testing-20180317.202121~258^2
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=57ecc1ef1c1d4df973517c71ba02e5d1a16fb9c1;p=ceph-ci.git

mon: mark manager beacons as no_reply

We never reply to manager beacons, and we have to mark them
that way or else forwarded messages pile up pending replies
and things eventually block.

Fixes: http://tracker.ceph.com/issues/22114
Reported-by: Hongpeng Lu <ludehp@163.com>

Signed-off-by: Greg Farnum <gfarnum@redhat.com>
---

diff --git a/src/mon/MgrMonitor.cc b/src/mon/MgrMonitor.cc
index 76985abc043..b9d79c3cf47 100644
--- a/src/mon/MgrMonitor.cc
+++ b/src/mon/MgrMonitor.cc
@@ -267,6 +267,7 @@ public:
 bool MgrMonitor::preprocess_beacon(MonOpRequestRef op)
 {
   MMgrBeacon *m = static_cast<MMgrBeacon*>(op->get_req());
+  mon->no_reply(op); // we never reply to beacons
   dout(4) << "beacon from " << m->get_gid() << dendl;
 
   if (!check_caps(op, m->get_fsid())) {