On 18/07/17 08:28, Justin Pettit wrote:
On Jul 17, 2017, at 10:06 PM, Numan Siddique <[email protected]> wrote:
On Tue, Jul 18, 2017 at 9:42 AM, Justin Pettit <[email protected]> wrote:
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:
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.
Reverting the patch and a release 2.7.2 sounds fine to me.
Are you okay with that, Eelco? Unless anyone objects, we'll leave the commit
on master and look at applying your new patch.
--Justin
Sound fine to me.
//Eelco
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
