Skip to content

Commit

Permalink
ovn-controller: Omit alert for Port_Binding.external_ids changes
Browse files Browse the repository at this point in the history
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 <nusiddiq@redhat.com>
Signed-off-by: 0-day Robot <robot@bytheb.org>
  • Loading branch information
numansiddique authored and ovsrobot committed Jun 24, 2019
1 parent 88d2ac5 commit 37a7c57
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions ovn/controller/ovn-controller.c
Expand Up @@ -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);
Expand Down

0 comments on commit 37a7c57

Please sign in to comment.