From a8a4d9565ba652b2f610ffad4e78d1f9068248ad Mon Sep 17 00:00:00 2001 From: Ben Pfaff Date: Thu, 8 Mar 2012 10:49:47 -0800 Subject: [PATCH] ovs-xapi-sync: Rerun processing when a db update arrives during a commit. The logic in ovs-xapi-sync didn't handle the case where ovsdb-server sends a database update before it replies to a transaction that ovs-xapi-sync sent, like this: ovs-xapi-sync ovsdb-server ------------- ------------ . . . transaction request ---> <--- database contents update <--- transaction reply . . . The update was not lost but ovs-xapi-sync would not process it until the database changed again. Bug #10082. Reported-by: Krishna Miriyala Signed-off-by: Ben Pfaff --- xenserver/usr_share_openvswitch_scripts_ovs-xapi-sync | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/xenserver/usr_share_openvswitch_scripts_ovs-xapi-sync b/xenserver/usr_share_openvswitch_scripts_ovs-xapi-sync index 7132726a..0ffccc2d 100755 --- a/xenserver/usr_share_openvswitch_scripts_ovs-xapi-sync +++ b/xenserver/usr_share_openvswitch_scripts_ovs-xapi-sync @@ -272,8 +272,10 @@ def main(): bridges = {} # Map from bridge name to xs_network_uuids iface_ids = {} # Map from xs-vif-uuid to iface-id vm_ids = {} # Map from xs-vm-uuid to vm-id + seqno = idl.change_seqno # Sequence number when we last processed the db while True: - if not force_run and not idl.run(): + idl.run() + if not force_run and seqno == idl.change_seqno: poller = ovs.poller.Poller() idl.wait(poller) poller.block() @@ -285,6 +287,7 @@ def main(): iface_ids = {} vm_ids = {} force_run = False + seqno = idl.change_seqno txn = ovs.db.idl.Transaction(idl) -- 2.30.2