On Fri, Jun 2, 2017 at 8:31 AM, <majop...@redhat.com> wrote: > From: Miguel Angel Ajo <majop...@redhat.com> > > This patch handles multiple gateways with priorities in chassisredirect > ports, any gateway with a chassis redirect port will implement the > rules to de-encapsulate incomming packets for such port. > > And hosts targetting a remote chassisredirect port will setup a > bundle(active_backup, ..) action to each tunnel port, in the given > priority order. > > Signed-off-by: Miguel Angel Ajo <majop...@redhat.com> > --- > ovn/controller/binding.c | 9 +-- > ovn/controller/lflow.c | 6 +- > ovn/controller/lport.c | 119 > ++++++++++++++++++++++++++++++++++++++++ > ovn/controller/lport.h | 28 ++++++++++ > ovn/controller/ovn-controller.c | 5 +- > ovn/controller/physical.c | 114 ++++++++++++++++++++++++++++++++------ > 6 files changed, 255 insertions(+), 26 deletions(-)
Some high level comments to start ... Ideally with a patch series, each patch should be applicable on its own. With this patch applied, some tests are failing for me. Documentation should also be included with whatever patch first introduces functionality, so I'd expect docs on the updated redirect-chassis format here. Please read over Documentation/internals/contributing/coding-style.rst. There are some minor style issues throughout the patch. I can point them out in a more detailed pass. The patch makes me wonder if we should introduce a more structured format for specifying chassis associated with a router port. It feels like we're encoding too much in a single option string. Maybe we should add a new "chassis" column to Logical_Router_Port, that can include a list of chassis, which would have to be a new record type in OVN northbound, containing much less info than the southbound counterpart. We'd have to add a similar new column to the Port_Binding table in OVN southbound. I'm curious what you and others think about this, or if the parsed option string is fine. -- Russell Bryant _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev