<p dir="ltr" id="yui_3_16_0_1_1415090525132_25078">Clearing all flows upon agent restart is a major issue, imho.  We should really look at this with higher priority than the modular L2 agent as the timeline of the refactor isn't clear for the modular layer 2 agent.  Whatever the issue was, I think we ought to be able to find a better solution that doesn't disrupt the network.  I agree that reconciling data after a restart is not straight-forward in all scenarios but there should be an option to just do basic sanity and not interrupt existing flows.  I'd like to help out on this (if needed) - there is a blueprint [1] that was suggested but I'm not sure who the owner is and what the status is.  If anyone is working on this and is at the summit this week, please let me know.  We can meet one of the days here at the summit.<br></p><p id="yui_3_16_0_1_1415090525132_27177" dir="ltr"><br></p><p id="yui_3_16_0_1_1415090525132_27178" dir="ltr">thanks,</p><p id="yui_3_16_0_1_1415090525132_27180" dir="ltr">manish<br></p><p id="yui_3_16_0_1_1415090525132_27155" dir="ltr"><br></p><p id="yui_3_16_0_1_1415090525132_27157" dir="ltr">[1] <a id="yui_3_16_0_1_1415090525132_27156" href="https://blueprints.launchpad.net/neutron/+spec/neutron-agent-soft-restart">Adding an option of "Soft Restart" in neutron agent along with o... : Blueprints : neutron</a><br><span></span></p><div style="width:450px; font-family: 'Georgia', 'Times', 'Times New Roman', 'serif';margin-top:5px; margin-bottom: 5px; background-color: #ffffff;" id="enhancrCard_0" class="link-enhancr-attachment link-enhancr-element" contenteditable="false"><table id="yui_3_16_0_1_1415090525132_30795" class="link-enhancr-element" style="width:450px; height:170px; position: relative; display: block;" border="0" cellpadding="0" cellspacing="0"><tbody id="yui_3_16_0_1_1415090525132_30794"><tr id="yui_3_16_0_1_1415090525132_33118" class="link-enhancr-element"><td id="yui_3_16_0_1_1415090525132_33117" class="link-enhancr-element" colspan="8" style="height: 1px; background-color: #e5e5e5; font-size: 1px; border-collapse: collapse;"><div id="yui_3_16_0_1_1415090525132_33116" class="link-enhancr-element" style="height: 1px; background-color: #e5e5e5; font-size: 1px; line-height:0px;"> </div></td></tr><tr id="yui_3_16_0_1_1415090525132_30793" class="link-enhancr-element"><td rowspan="5" class="link-enhancr-element" style="width: 1px; background-color: #e5e5e5; font-size: 1pt; border-collapse: collapse;"><div class="link-enhancr-element" style="width: 1px; background-color: #e5e5e5; font-size: 1pt;"> </div></td><td id="yui_3_16_0_1_1415090525132_30792" class="link-enhancr-element" rowspan="5" style="vertical-align: middle; width: 168px; height: 168px; background-color: #000000;border-collapse: collapse;"><div id="yui_3_16_0_1_1415090525132_30791" class="link-enhancr-element" style="width: 168px;" align="center"><a id="yui_3_16_0_1_1415090525132_30790" href="https://blueprints.launchpad.net/neutron/+spec/neutron-agent-soft-restart" class="link-enhancr-card-urlWrapper link-enhancr-element" style="text-decoration: none !important; color: #000000 !important;"><img id="yui_3_16_0_1_1415090525132_30789" alt="image" src="https://launchpadlibrarian.net/83674394/os64.png" class="link-enhancr-thumbnail-image link-enhancr-element" style="display: block; margin: auto;" height="168" width="168"></a></div></td><td rowspan="5" class="link-enhancr-element" style="width: 1px; background-color: #e5e5e5; font-size: 0pt; border-collapse: collapse;"><div class="link-enhancr-element" style="width: 1px; background-color: #e5e5e5; font-size: 1pt;"> </div></td><td rowspan="5" class="link-enhancr-element" style="width: 14px; background-color: #ffffff; font-size: 0pt; border-collapse: collapse;"><div class="link-enhancr-element" style="width: 14px; background-color: #ffffff; font-size: 14pt;"> </div></td><td colspan="2" class="link-enhancr-element" style="height: 6px; background-color: #ffffff; font-size: 0pt; border-collapse: collapse;"><div class="link-enhancr-element" style="height: 6px; background-color: #ffffff; font-size: 6pt;"> </div></td><td rowspan="5" class="link-enhancr-element" style="width: 20px; background-color: #ffffff; font-size: 0pt; border-collapse: collapse;"><div class="link-enhancr-element" style="width: 20px; background-color: #ffffff; font-size: 20pt;"> </div></td><td class="link-enhancr-element" rowspan="5" style="width: 1px; background-color: #e5e5e5; font-size: 1pt; border-collapse: collapse;" width="1"><div class="link-enhancr-element" style="width: 1px; background-color: #e5e5e5; font-size: 1pt;"> </div></td></tr><tr id="yui_3_16_0_1_1415090525132_33124"><td id="yui_3_16_0_1_1415090525132_33123" class="link-enhancr-element" colspan="2" style="width: 100%; vertical-align: middle; font-family: 'Georgia', 'Times', 'Times New Roman', 'serif';"><div id="yui_3_16_0_1_1415090525132_33122" class="link-enhancr-text-part link-enhancr-element" style="line-height:16.5px; background-color: #ffffff; height: 135px; width: 245px;"><div id="yui_3_16_0_1_1415090525132_33121" class="link-enhancr-element" style="word-wrap: break-word; word-break: break-all;"><span class="link-enhancr-element icon  icon-shrink link-enhancr-toggle"></span><span class="link-enhancr-element icon icon-close link-enhancr-delete"></span><a id="yui_3_16_0_1_1415090525132_33120" href="https://blueprints.launchpad.net/neutron/+spec/neutron-agent-soft-restart" class="link-enhancr-card-urlWrapper link-enhancr-element" style="text-decoration: none !important; color: #000000 !important; line-height: 100%; font-size: 18px; display: block;"><span id="yui_3_16_0_1_1415090525132_33119" class="link-enhancr-element link-enhancr-card-title" style="margin: 0; font-weight: normal;margin-bottom: 3px; font-size: 18px; line-height: 21px; max-height: 43px; color: #000000; overflow: hidden !important; display: inline-block;">Adding an option of "Soft Restart" in neutron agent alon...</span></a><div style="font-size: 13px; line-height: 20px; color: #999999; max-height: 81px; font-family: 'Georgia', 'Times', 'Times New Roman', 'serif';overflow: hidden;" class="link-enhancr-card-description link-enhancr-element">While the blueprint of "ovs-firewall-driver" is being developed, a new concern comes up. When an ovs agent (or an ml2 agent with ovs) restarts, if it cleans up all ...</div></div></div></td></tr><tr><td colspan="2" class="link-enhancr-element" style="height: 4px; background-color: #ffffff; font-size: 0pt; border-collapse: collapse;"><div class="link-enhancr-element" style="height: 4px; background-color: #ffffff; font-size: 4pt;"></div></td></tr><tr><td class="link-enhancr-element" style="vertical-align: middle; font-family: 'Arial', 'Helvetica Neue', 'Helvetica', 'sans-serif';"><div class="link-enhancr-element" style="font-size: 0pt;"><a href="https://blueprints.launchpad.net/neutron/+spec/neutron-agent-soft-restart" class="link-enhancr-card-url link-enhancr-element" style="color: black; text-decoration: none !important;cursor:pointer !important;" target="_blank"><span id="yui_3_16_0_1_1415090525132_27173" class="link-enhancr-element link-enhancr-view-on" style="display: inline-block; line-height: 11px; max-width: 145px; min-width: 85px; overflow: hidden; max-height: 13px; word-break: break-all;"><span class="link-enhancr-element link-enhancr-mobile-no-resize" style="vertical-align:middle; font-size: 9px; line-height: 11px; color: #999999; -moz-text-size-adjust: none; -ms-text-size-adjust: none; -webkit-text-size-adjust:none; text-size-adjust:none;">View on <span style="font-weight: bold" class="link-enhancr-view-on-domain">blueprints.launchpad.net</span></span></span></a></div></td><td class="link-enhancr-element" style="vertical-align: middle; width: 100px; font-family: 'Arial', 'Helvetica Neue', 'Helvetica', 'sans-serif';"><div class="link-enhancr-element link-enhancr-preview-wrapper" style="max-width: 100px; min-width: 80px; overflow: hidden; text-align: right; line-height: 11px; max-height: 13px; font-size: 0pt;"><span class="link-enhancr-element link-enhancr-preview-by link-enhancr-mobile-no-resize" style="vertical-align:middle; font-size: 9px; line-height: 11px; color: #999999; -moz-text-size-adjust: none; -ms-text-size-adjust: none; -webkit-text-size-adjust:none; text-size-adjust:none;">Preview by Yahoo</span></div></td></tr><tr><td colspan="2" class="link-enhancr-element" style="height: 9px; background-color: #ffffff; font-size: 0pt; border-collapse: collapse;"><div class="link-enhancr-element" style="height: 9px; background-color: #ffffff; font-size: 9pt;"></div></td></tr><tr class="link-enhancr-element"><td class="link-enhancr-element" colspan="8" style="height: 1px; background-color: #e5e5e5; font-size: 1px; border-collapse: collapse;"><div class="link-enhancr-element" style="height: 1px; background-color: #e5e5e5; font-size: 1px; line-height:0px"> </div></td></tr></tbody></table></div><p id="yui_3_16_0_1_1415090525132_27157">  </p><div style="display: none;" class="yahoo_quoted"> <div style="font-family: HelveticaNeue, Helvetica Neue, Helvetica, Arial, Lucida Grande, sans-serif; font-size: 16px;"> <div style="font-family: HelveticaNeue, Helvetica Neue, Helvetica, Arial, Lucida Grande, sans-serif; font-size: 16px;"> <div dir="ltr"> <font face="Arial" size="2"> On Friday, October 31, 2014 7:32 AM, Ben Nemec <openstack@nemebean.com> wrote:<br> </font> </div>  <br><br> <div class="y_msg_container">On 10/29/2014 10:17 AM, Kyle Mestery wrote:<br clear="none">> On Wed, Oct 29, 2014 at 7:25 AM, Hly <<a shape="rect" ymailto="mailto:henry4hly@gmail.com" href="mailto:henry4hly@gmail.com">henry4hly@gmail.com</a>> wrote:<br clear="none">>><br clear="none">>><br clear="none">>> Sent from my iPad<br clear="none">>><br clear="none">>> On 2014-10-29, at 下午8:01, Robert van Leeuwen <<a shape="rect" ymailto="mailto:Robert.vanLeeuwen@spilgames.com" href="mailto:Robert.vanLeeuwen@spilgames.com">Robert.vanLeeuwen@spilgames.com</a>> wrote:<br clear="none">>><br clear="none">>>>>> I find our current design is remove all flows then add flow by entry, this<br clear="none">>>>>> will cause every network node will break off all tunnels between other<br clear="none">>>>>> network node and all compute node.<br clear="none">>>>> Perhaps a way around this would be to add a flag on agent startup<br clear="none">>>>> which would have it skip reprogramming flows. This could be used for<br clear="none">>>>> the upgrade case.<br clear="none">>>><br clear="none">>>> I hit the same issue last week and filed a bug here:<br clear="none">>>> <a shape="rect" href="https://bugs.launchpad.net/neutron/+bug/1383674" target="_blank">https://bugs.launchpad.net/neutron/+bug/1383674</a><br clear="none">>>><br clear="none">>>> 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 clear="none">>>> e.g. something simple like changing a log setting becomes a hassle.<br clear="none">>>> I would prefer the default behaviour to be to not clear the flows or at the least an config option to disable it.<br clear="none">>>><br clear="none">>><br clear="none">>> +1, we also suffered from this even when a very little patch is done<br clear="none">>><br clear="none">> I'd really like to get some input from the tripleo folks, because they<br clear="none">> were the ones who filed the original bug here and were hit by the<br clear="none">> agent NOT reprogramming flows on agent restart. It does seem fairly<br clear="none">> obvious that adding an option around this would be a good way forward,<br clear="none">> however.<br clear="none"><br clear="none">Since nobody else has commented, I'll put in my two cents (though I<br clear="none">might be overcharging you ;-).  I've also added the TripleO tag to the<br clear="none">subject, although with Summit coming up I don't know if that will help.<br clear="none"><br clear="none">Anyway, if the bug you're referring to is the one I think, then our<br clear="none">issue was just with the flows not existing.  I don't think we care<br clear="none">whether they get reprogrammed on agent restart or not as long as they<br clear="none">somehow come into existence at some point.<br clear="none"><br clear="none">It's possible I'm wrong about that, and probably the best person to talk<br clear="none">to would be Robert Collins since I think he's the one who actually<br clear="none">tracked down the problem in the first place.<br clear="none"><br clear="none">-Ben<div class="yqt0784029500" id="yqtfd97801"><br clear="none"><br clear="none"><br clear="none">_______________________________________________<br clear="none">OpenStack-dev mailing list<br clear="none"><a shape="rect" ymailto="mailto:OpenStack-dev@lists.openstack.org" href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br clear="none"><a shape="rect" 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 clear="none"></div><br><br></div>  </div> </div>  </div>