Re: [ovs-dev] [PATCH branch 2.12 2/2] ovn-controller: Fix IP engine run with in-flight messages

2019-08-08 Thread Justin Pettit
Thanks. I added a reference to the OVN repo commit and merged this into branch-2.12. --Justin > On Aug 5, 2019, at 8:50 AM, Han Zhou wrote: > > From: Dumitru Ceara > > When trying to incrementally process changes even if there are in-flight > messages we were incorrectly setting the

Re: [ovs-dev] [PATCH branch 2.12 2/2] ovn-controller: Fix IP engine run with in-flight messages

2019-08-05 Thread 0-day Robot
Bleep bloop. Greetings Han Zhou, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: Unexpected sign-offs from developers who are not authors or co-authors or committers:

[ovs-dev] [PATCH branch 2.12 2/2] ovn-controller: Fix IP engine run with in-flight messages

2019-08-05 Thread Han Zhou
From: Dumitru Ceara When trying to incrementally process changes even if there are in-flight messages we were incorrectly setting the engine_aborted variable to the value returned by engine_run. However, engine_run returns true if the run was completed. This was causing discrepancies between