summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSage Weil <sage@inktank.com>2013-07-18 14:46:57 -0700
committerGreg Farnum <greg@inktank.com>2013-07-19 13:21:48 -0700
commit000d4d38a47871002a870bc815893833dc5a5773 (patch)
tree1f3986995aca69a91f1e53e1f244e752f3774e5a
parent30de04066d874921b7d721bfcb0f363df8a735f1 (diff)
downloadceph-000d4d38a47871002a870bc815893833dc5a5773.tar.gz
mon: mark_down session by con, not addr
We have the ConnectionRef here; use it. This avoids generating a spurious RESET event for the connection. Signed-off-by: Sage Weil <sage@inktank.com>
-rw-r--r--src/mon/Monitor.cc10
1 files changed, 5 insertions, 5 deletions
diff --git a/src/mon/Monitor.cc b/src/mon/Monitor.cc
index 3ff0a6418b2..90750dd7b11 100644
--- a/src/mon/Monitor.cc
+++ b/src/mon/Monitor.cc
@@ -3477,16 +3477,16 @@ void Monitor::tick()
continue;
if (!s->until.is_zero() && s->until < now) {
- dout(10) << " trimming session " << s->inst
+ dout(10) << " trimming session " << s->con << " " << s->inst
<< " (until " << s->until << " < now " << now << ")" << dendl;
- messenger->mark_down(s->inst.addr);
+ messenger->mark_down(s->con);
remove_session(s);
} else if (!exited_quorum.is_zero()) {
if (now > (exited_quorum + 2 * g_conf->mon_lease)) {
// boot the client Session because we've taken too long getting back in
- dout(10) << " trimming session " << s->inst
- << " because we've been out of quorum too long" << dendl;
- messenger->mark_down(s->inst.addr);
+ dout(10) << " trimming session " << s->con << " " << s->inst
+ << " because we've been out of quorum too long" << dendl;
+ messenger->mark_down(s->con);
remove_session(s);
}
}