]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
mon/MonClient: drop global_id arg from _add_conn() and _add_conns()
authorIlya Dryomov <idryomov@gmail.com>
Tue, 30 Mar 2021 13:19:41 +0000 (15:19 +0200)
committerIlya Dryomov <idryomov@gmail.com>
Mon, 12 Apr 2021 18:50:53 +0000 (20:50 +0200)
Passing anything but MonClient instance's global_id doesn't make
sense.

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
(cherry picked from commit a71f6e90d43cca5a79db92ca6a640598796ae7ee)

src/mon/MonClient.cc
src/mon/MonClient.h

index 3fa40e30d891f25b8e729c844be0aee4b95eef3e..f5cbc2a8052e67bf046ff1e1d0020dbf4a3d3be3 100644 (file)
@@ -725,9 +725,9 @@ void MonClient::_reopen_session(int rank)
   _start_hunting();
 
   if (rank >= 0) {
-    _add_conn(rank, global_id);
+    _add_conn(rank);
   } else {
-    _add_conns(global_id);
+    _add_conns();
   }
 
   // throw out old queued messages
@@ -749,7 +749,7 @@ void MonClient::_reopen_session(int rank)
   }
 }
 
-void MonClient::_add_conn(unsigned rank, uint64_t global_id)
+void MonClient::_add_conn(unsigned rank)
 {
   auto peer = monmap.get_addrs(rank);
   auto conn = messenger->connect_to_mon(peer);
@@ -764,7 +764,7 @@ void MonClient::_add_conn(unsigned rank, uint64_t global_id)
                  << dendl;
 }
 
-void MonClient::_add_conns(uint64_t global_id)
+void MonClient::_add_conns()
 {
   // collect the next batch of candidates who are listed right next to the ones
   // already tried
@@ -817,7 +817,7 @@ void MonClient::_add_conns(uint64_t global_id)
     n = ranks.size();
   }
   for (unsigned i = 0; i < n; i++) {
-    _add_conn(ranks[i], global_id);
+    _add_conn(ranks[i]);
     tried.insert(ranks[i]);
   }
 }
index cc7805d0675e9bb852979ed18ea26de7f5dc1265..cb16965450bf27f237828fccb99bf4da778ad309 100644 (file)
@@ -351,9 +351,9 @@ private:
   void _finish_hunting(int auth_err);
   void _finish_auth(int auth_err);
   void _reopen_session(int rank = -1);
-  void _add_conn(unsigned rank, uint64_t global_id);
+  void _add_conn(unsigned rank);
+  void _add_conns();
   void _un_backoff();
-  void _add_conns(uint64_t global_id);
   void _send_mon_message(MessageRef m);
 
   std::map<entity_addrvec_t, MonConnection>::iterator _find_pending_con(