]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commitdiff
Merge pull request #43106 from rzarzynski/wip-crimson-msgr-bind
authorSamuel Just <sjust@redhat.com>
Wed, 15 Sep 2021 04:41:46 +0000 (21:41 -0700)
committerGitHub <noreply@github.com>
Wed, 15 Sep 2021 04:41:46 +0000 (21:41 -0700)
crimson: add support for ms_bind_retry_{delay,count}

Reviewed-by: Yingxin Cheng <yingxin.cheng@intel.com>
1  2 
src/crimson/common/errorator.h
src/crimson/net/SocketMessenger.cc

Simple merge
index 6b5dd19f1e798956342e0c07c1444c3068a7d965,c688bd139fc3bb3a79594690e9f354bf10645830..e83f2473ebd134f6d742461762afc5124438ed94
@@@ -88,13 -81,9 +81,13 @@@ SocketMessenger::bind_ertr::future<
  SocketMessenger::try_bind(const entity_addrvec_t& addrs,
                            uint32_t min_port, uint32_t max_port)
  {
 -  auto addr = addrs.front();
 +  // the classical OSD iterates over the addrvec and tries to listen on each
 +  // addr. crimson doesn't need to follow as there is a consensus we need to
 +  // worry only about proto v2.
 +  assert(addrs.size() == 1);
 +  auto addr = addrs.msgr2_addr();
    if (addr.get_port() != 0) {
-     return do_bind(addrs).safe_then([this] {
+     return do_listen(addrs).safe_then([this] {
        logger().info("{} try_bind: done", *this);
      });
    }