On Fri, Dec 11, 2020 at 01:36:25PM +0100, Dumitru Ceara wrote: > On 12/2/20 7:50 AM, Ben Pfaff wrote: > > Many of these could be replaced by "ovn-nbctl sync". Some weren't > > really needed at all because they were adjacent to something that itself > > called sync or otherwise used --wait. Some were more appropriately > > done with explicit waits for what was really needed. > > > > I left some "sleep"s. Some were because they were "negative" sleeps: > > they were giving time for something to happen that should *not* happen > > (in other words, if you wait for it to happen, you'll wait forever, > > unless there's a bug). Some were because I didn't know how to properly > > wait for what they were waiting for, or because I didn't understand > > the circumstances deeply enough. > > > > Signed-off-by: Ben Pfaff <b...@ovn.org> > > --- > > Hi Ben, > > I have a tiny nit below, with that addressed: > > Acked-by: Dumitru Ceara <dce...@redhat.com>
I made that change and applied this patch to master. Thanks! _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev