On Thu, Jun 03, 2021 at 07:38:01PM -0400, Numan Siddique wrote:
> On Thu, Jun 3, 2021 at 3:48 PM Ben Pfaff <[email protected]> wrote:
> >
> > Utilities like ovn-sbctl sometimes need to retry their transactions
> > because of races.  For this reason, instead of sending user output
> > directly to stdout, they buffer it until the transaction succeeds.
> > Some of the ovn-sbctl commands didn't do this properly, so they would
> > output multiple times upon a race.  Another way to see the problem
> > was to use daemon mode, in which the output written directly with
> > printf() would not appear at all, since the daemon's stdout is not
> > connected to ovn-sbctl's stdout.
> >
> > Signed-off-by: Ben Pfaff <[email protected]>
> > Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=1965780
> > Reported-by: Alexey Roytman <[email protected]>
> 
> Acked-by: Numan Siddique <[email protected]>

Thanks, I applied this to master and branch-21.06.

In theory this could be backported further, because it would also
manifest upon races, but I don't think that's too likely because
ovn-sbctl is rarely used to both modify things and dump flows in a
single transaction.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to