<div dir="ltr">Hi Salvatore,<div>A startup flag is really a simpler approach. But in what situation we should set this flag to remove all flows? upgrade? restart manually? internal fault?</div><div><br></div><div>Indeed, only at the time that there are inconsistent(incorrect, unwanted, stable and so on) flows between agent and the ovs related, we need refresh flows. But the problem is how we know this? I think a startup flag is too rough, unless we can tolerate the inconsistent situation.</div><div><br></div><div>Of course, I believe that turn off startup reset flows action can resolve most problem. The flows are correct most time after all. But considering NFV 5 9s, I still recommend <span style="font-family:arial,sans-serif;font-size:12.8000001907349px">flow synchronization approach.</span></div><div><br></div><div>BR,</div><div>Germy</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 5, 2014 at 3:36 PM, Salvatore Orlando <span dir="ltr"><<a href="mailto:sorlando@nicira.com" target="_blank">sorlando@nicira.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">From what I gather from this thread and related bug report, the change introduced in the OVS agent is causing a data plane outage upon agent restart, which is not desirable in most cases.<div><br></div><div>The rationale for the change that introduced this bug was, I believe, cleaning up stale flows on the OVS agent, which also makes some sense.</div><div><br></div><div>Unless I'm missing something, I reckon the best way forward is actually quite straightforward; we might add a startup flag to reset all flows and not reset them by default.</div><div>While I agree the "flow synchronisation" process proposed in the previous post is valuable too, I hope we might be able to fix this with a simpler approach.</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>Salvatore</div></font></span></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On 5 November 2014 04:43, Germy Lure <span dir="ltr"><<a href="mailto:germy.lure@gmail.com" target="_blank">germy.lure@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi,<div><br></div><div>Consider the triggering of restart agent, I think it's nothing but:</div><div>1). only restart agent</div><div>2). reboot the host that agent deployed on</div><div><br></div><div>When the agent started, the ovs may:</div><div>a.have all correct flows</div><div>b.have nothing at all</div><div>c.have partly correct flows, the others may need to be reprogrammed, deleted or added</div><div><br></div><div>In any case, I think both user and developer would happy to see that the system recovery ASAP after agent restarting. The best is agent only push those incorrect flows, but keep the correct ones. This can ensure those business with correct flows working during agent starting.</div><div><br></div><div>So, I suggest two solutions:</div><div>1.Agent gets all flows from ovs and compare with its local flows after restarting. And agent only corrects the different ones.</div><div>2.Adapt ovs and agent. Agent just push all(not remove) flows every time and ovs prepares two tables for flows switch(like RCU lock).</div><div><br></div><div>1 is recommended because of the 3rd vendors.</div><div><br></div><div>BR,</div><div>Germy</div><div><br></div></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 31, 2014 at 10:28 PM, Ben Nemec <span dir="ltr"><<a href="mailto:openstack@nemebean.com" target="_blank">openstack@nemebean.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On 10/29/2014 10:17 AM, Kyle Mestery wrote:<br>
> On Wed, Oct 29, 2014 at 7:25 AM, Hly <<a href="mailto:henry4hly@gmail.com" target="_blank">henry4hly@gmail.com</a>> wrote:<br>
>><br>
>><br>
>> Sent from my iPad<br>
>><br>
>> On 2014-10-29, at 下午8:01, Robert van Leeuwen <<a href="mailto:Robert.vanLeeuwen@spilgames.com" target="_blank">Robert.vanLeeuwen@spilgames.com</a>> wrote:<br>
>><br>
>>>>> I find our current design is remove all flows then add flow by entry, this<br>
>>>>> will cause every network node will break off all tunnels between other<br>
>>>>> network node and all compute node.<br>
>>>> Perhaps a way around this would be to add a flag on agent startup<br>
>>>> which would have it skip reprogramming flows. This could be used for<br>
>>>> the upgrade case.<br>
>>><br>
>>> I hit the same issue last week and filed a bug here:<br>
>>> <a href="https://bugs.launchpad.net/neutron/+bug/1383674" target="_blank">https://bugs.launchpad.net/neutron/+bug/1383674</a><br>
>>><br>
>>> From an operators perspective this is VERY annoying since you also cannot push any config changes that requires/triggers a restart of the agent.<br>
>>> e.g. something simple like changing a log setting becomes a hassle.<br>
>>> I would prefer the default behaviour to be to not clear the flows or at the least an config option to disable it.<br>
>>><br>
>><br>
>> +1, we also suffered from this even when a very little patch is done<br>
>><br>
> I'd really like to get some input from the tripleo folks, because they<br>
> were the ones who filed the original bug here and were hit by the<br>
> agent NOT reprogramming flows on agent restart. It does seem fairly<br>
> obvious that adding an option around this would be a good way forward,<br>
> however.<br>
<br>
</span>Since nobody else has commented, I'll put in my two cents (though I<br>
might be overcharging you ;-).  I've also added the TripleO tag to the<br>
subject, although with Summit coming up I don't know if that will help.<br>
<br>
Anyway, if the bug you're referring to is the one I think, then our<br>
issue was just with the flows not existing.  I don't think we care<br>
whether they get reprogrammed on agent restart or not as long as they<br>
somehow come into existence at some point.<br>
<br>
It's possible I'm wrong about that, and probably the best person to talk<br>
to would be Robert Collins since I think he's the one who actually<br>
tracked down the problem in the first place.<br>
<span><font color="#888888"><br>
-Ben<br>
</font></span><div><div><br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org" target="_blank">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br></div>
</div></div><br>_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org" target="_blank">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></blockquote></div><br></div>
</div></div><br>_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></blockquote></div><br></div>