> On Jul 17, 2017, at 8:51 PM, Guru Shetty <[email protected]> wrote: > > >> >> On 17 July 2017 at 12:51, Justin Pettit <[email protected]> wrote: >> >> > On Jul 17, 2017, at 7:49 AM, Eelco Chaudron <[email protected]> wrote: >> > >> > On 17/07/17 11:10, Numan Siddique wrote: >> >> >> >> >> >> On Mon, Jul 17, 2017 at 1:10 PM, Eelco Chaudron <[email protected] >> >> <mailto:[email protected]>> wrote: >> >> >> >> Assuming we can get the patch in before the next release, there >> >> should be no need to revert back. But if its common practice for >> >> OVS to do it right away I have no issue with it. >> >> >> >> >> >> For tripleo, we need OVS 2.7.1 for OVN DB HA support, but we can't take >> >> v2.7.1 for this issue. I think we should atleast revert the patch for 2.7 >> >> branch and have a new version 2.7.2. >> >> >> > If we are having a 2.7.2 release specially for this it makes sense to >> > revert it. >> > >> > In addition can you test/apply the updated patch, >> > https://mail.openvswitch.org/pipermail/ovs-dev/2017-July/335560.html as I >> > was hitting the issue with the persistant ports setup, it would be good to >> > see your case is also fixed. >> >> Han and Numan, it sounds like Eelco's original patch fixed an issue, but >> it's introducing problems for more common workloads. Can you try it on the >> tip of "branch-2.7" and see if it resolves the issues you're seeing? I'd >> suggest if it doesn't fix the problems, we revert Eelco's orignal patch and >> release 2.7.2. If it does, and people think it's reasonably safe, we can >> release 2.7.2 with the patches. > > Let me know if anyone has a different view on how we should proceed. > > IMO, we should just revert this patch for 2.7 branch till we have a proper > fix. If we apply another patch in haste to fix this issue without letting it > bake for more testing and immediately release 2.7.2, we may just end up with > a different bug. > > I tested the revert on tip of branch-2.7 and it atleast fixes the issue where > geneve tunnels across 2 hosts start to work again and I also do not see the > "open failed (File exists))" error too.
Sounds reasonable to me. Does anyone else have any opinions? If I don't hear anything by late tomorrow morning (PDT), I'll revert the patch and release 2.7.2. Thanks, --Justin _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
