[ovs-dev] [PATCH v2] ovn: Support l2gateway-chassis option in "l2gateway" logical ports

Russell Bryant russell at ovn.org
Fri Jul 8 15:00:40 UTC 2016


On Fri, Jul 8, 2016 at 6:37 AM, Numan Siddique <nusiddiq at redhat.com> wrote:

> ovn-controller will now bind the l2gateway logical ports.
>
> Signed-Off-by: Numan Siddique <nusiddiq at redhat.com>
>

Thanks!  I applied this to master with one very minor style tweak:

diff --git a/ovn/controller/binding.c b/ovn/controller/binding.c
index 3061b34..4704226 100644
--- a/ovn/controller/binding.c
+++ b/ovn/controller/binding.c
@@ -226,7 +226,7 @@ consider_local_datapath(struct controller_ctx *ctx,
struct shash *lports,
         const char *chassis_id = smap_get(&binding_rec->options,
                                           "l2gateway-chassis");
         if (!chassis_id || strcmp(chassis_id, chassis_rec->name)) {
-            if ((binding_rec->chassis == chassis_rec) &&
ctx->ovnsb_idl_txn) {
+            if (binding_rec->chassis == chassis_rec && ctx->ovnsb_idl_txn)
{
                 VLOG_INFO("Releasing l2gateway port %s from this chassis.",
                           binding_rec->logical_port);
                 sbrec_port_binding_set_chassis(binding_rec, NULL);



-- 
Russell Bryant



More information about the dev mailing list