rabbitmq / rabbitmq-federation

RabbitMQ Federation plugin
https://www.rabbitmq.com/
Other
40 stars 21 forks source link

Handle operation timeouts during internal exchange checks #107

Closed michaelklishin closed 4 years ago

michaelklishin commented 4 years ago

Handle more failures around one-off connection calls

Timeouts were not handled gracefully which could disrupt the link for no good reason. Such one-off checks are meant to be best effort and not performed particularly frequently. Their failures should result in visible log messages but not link restarts.

While at it, improve logging in related code paths.

Per discussion with @kjnilsson.

michaelklishin commented 4 years ago

Backported to v3.8.x and v3.7.x. I also backported 7c0a784c70dc61f7f6433fccb3e857298038ce71 to v3.7.x which was a follow-up to 5129d6a91055ecbdd4ee7d2d92554f19046e3359 which was never backported to 3.7.