This was dropped in
bf4938567943c80345966f9c5a3bdc75a913175b but should
not have been. Since we are advertising the addr and not the bind
addr there is no reason to disable this check.
Signed-off-by: Sage Weil <sage@redhat.com>
<< " - presumably this is the same node!" << dendl;
} else {
ldout(async_msgr->cct, 10) << __func__ << " connect claims to be "
- << paddr << " not " << peer_addr
- << " (peer is possibly using public_bind_addr?) " << dendl;
+ << paddr << " not " << peer_addr << dendl;
+ goto fail;
}
}
<< paddr << " not " << peer_addr << " - presumably this is the same node!" << dendl;
} else {
ldout(msgr->cct,10) << "connect claims to be "
- << paddr << " not " << peer_addr
- << " (peer is possibly using public_bind_addr?) " << dendl;
+ << paddr << " not " << peer_addr << dendl;
+ goto fail;
}
}