Use a combination of cluster name and virtual host when detecting cycles

rabbitmq / rabbitmq-federation

RabbitMQ Federation plugin

https://www.rabbitmq.com/

Other

40 stars 21 forks source link

Use a combination of cluster name and virtual host when detecting cycles #113

Closed michaelklishin closed 4 years ago

michaelklishin commented 4 years ago

This makes it possible to federate messages between more than two virtual hosts in a single cluster. Previously cycle detection would drop messages that have traversed a "hop" (federation link) in the same cluster.

Closes #116.

lukebakken commented 4 years ago

@michaelklishin reviewing and testing now. I'm assuming this is no longer a WIP?

michaelklishin commented 4 years ago

I have to add a test which is moderately complex. Binding propagation also need updating.

lukebakken commented 4 years ago

@michaelklishin I have re-confirmed that these changes address the customer issue (VESC-984).

lukebakken commented 4 years ago

NOTE: when back-porting include #117

michaelklishin commented 4 years ago

Backported to v3.8.x.