diff mbox series

[ovs-dev,2/2] ovs-vsctl.c: Do not sent 'set_db_change_aware'.

Message ID 20230111000756.4054163-2-hzhou@ovn.org
State Superseded
Headers show
Series [ovs-dev,1/2] ovsdb-idl: Provide API to disable set_db_change_aware request. | expand

Checks

Context Check Description
ovsrobot/apply-robot success apply and check: success
ovsrobot/github-robot-_Build_and_Test fail github build: failed
ovsrobot/intel-ovs-compilation success test: success

Commit Message

Han Zhou Jan. 11, 2023, 12:07 a.m. UTC
ovs-vsctl's connections are short-lived, so it doesn't care about db
status changes.

Reported-by: Tobias Hofmann <tohofman@cisco.com>
Reported-at: https://mail.openvswitch.org/pipermail/ovs-discuss/2021-February/050914.html
Signed-off-by: Han Zhou <hzhou@ovn.org>
---
 utilities/ovs-vsctl.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Dumitru Ceara Jan. 11, 2023, 9:42 a.m. UTC | #1
On 1/11/23 01:07, Han Zhou wrote:
> ovs-vsctl's connections are short-lived, so it doesn't care about db
> status changes.
> 
> Reported-by: Tobias Hofmann <tohofman@cisco.com>
> Reported-at: https://mail.openvswitch.org/pipermail/ovs-discuss/2021-February/050914.html
> Signed-off-by: Han Zhou <hzhou@ovn.org>
> ---

Makes sense.

Acked-by: Dumitru Ceara <dceara@redhat.com>

Thanks,
Dumitru
diff mbox series

Patch

diff --git a/utilities/ovs-vsctl.c b/utilities/ovs-vsctl.c
index c1d470006169..2f5ac1a26225 100644
--- a/utilities/ovs-vsctl.c
+++ b/utilities/ovs-vsctl.c
@@ -180,6 +180,7 @@  main(int argc, char *argv[])
     ovsdb_idl_set_shuffle_remotes(idl, shuffle_remotes);
     ovsdb_idl_set_remote(idl, db, retry);
     ovsdb_idl_set_leader_only(idl, leader_only);
+    ovsdb_idl_set_db_change_aware(idl, false);
     run_prerequisites(commands, n_commands, idl);
 
     /* Execute the commands.