On Mon, Jun 24, 2019 at 3:59 AM <[email protected]> wrote: > > From: Numan Siddique <[email protected]> > > Running the command "ovn-nbctl set logical_switch_port foo external_ids:foo=bar" > results in the incremetal processing engine to recompute the flows on the > chassis where the logical port 'foo' is claimed. > > This patch avoids this unnecessary recomputation by omitting the alert for the > Port_Binding.external_ids column. > > Signed-off-by: Numan Siddique <[email protected]> > --- > ovn/controller/ovn-controller.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/ovn/controller/ovn-controller.c b/ovn/controller/ovn-controller.c > index 60190161f..ddfd05c71 100644 > --- a/ovn/controller/ovn-controller.c > +++ b/ovn/controller/ovn-controller.c > @@ -1732,6 +1732,8 @@ main(int argc, char *argv[]) > > ovsdb_idl_track_add_all(ovnsb_idl_loop.idl); > ovsdb_idl_omit_alert(ovnsb_idl_loop.idl, &sbrec_chassis_col_nb_cfg); > + ovsdb_idl_omit_alert(ovnsb_idl_loop.idl, > + &sbrec_port_binding_col_external_ids); > update_sb_monitors(ovnsb_idl_loop.idl, NULL, NULL, NULL); > > stopwatch_create(CONTROLLER_LOOP_STOPWATCH_NAME, SW_MS); > -- > 2.21.0 > > _______________________________________________ > dev mailing list > [email protected] > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Thanks for the improvement! Acked-by: Han Zhou <[email protected]> _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
