Skip to content

Commit

Permalink
mds: remove MonClient reconnect
Browse files Browse the repository at this point in the history
We should instead rely on the keepalives and automatic reconnects by the
MonClient.

Fixes: http://tracker.ceph.com/issues/36493

Signed-off-by: Patrick Donnelly <[email protected]>
  • Loading branch information
batrick committed Oct 17, 2018
1 parent 1d59e1e commit d8925ba
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 9 deletions.
8 changes: 0 additions & 8 deletions src/mds/Beacon.cc
Original file line number Diff line number Diff line change
Expand Up @@ -260,14 +260,6 @@ bool Beacon::is_laggy()
<< "s since last acked beacon" << dendl;
}
laggy = true;
auto last_reconnect = std::chrono::duration<double>(now-last_mon_reconnect).count();
if (since > (g_conf()->mds_beacon_grace*2) && last_reconnect > g_conf()->mds_beacon_interval) {
// maybe it's not us?
dout(1) << "initiating monitor reconnect; maybe we're not the slow one"
<< dendl;
last_mon_reconnect = now;
monc->reopen_session();
}
return true;
}
return false;
Expand Down
1 change: 0 additions & 1 deletion src/mds/Beacon.h
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,6 @@ class Beacon : public Dispatcher
version_t last_seq = 0; // last seq sent to monitor
std::map<version_t,time> seq_stamp; // seq # -> time sent
time last_acked_stamp = clock::zero(); // last time we sent a beacon that got acked
time last_mon_reconnect = clock::zero();
bool laggy = false;
time last_laggy = clock::zero();

Expand Down

0 comments on commit d8925ba

Please sign in to comment.