<div dir="ltr"><div class="gmail_extra">







<p class="">After review <a href="https://review.openstack.org/#/c/103280/"><span class="">https://review.openstack.org/#/c/103280/</span></a> I have following thoughts. This patch looks fine, but huge and has lot of changes in deployment logic. Parallel teams work (vmware, melanox) depends on current implementation of Neutron in FUEL. I have a some dreads, that they can’t refactor his code in the short time remaining.</p>


<p class="">Implementation <a href="https://review.openstack.org/#/c/103280">https://review.openstack.org/#/c/103280</a> has considerably less changes in current code of FUEL. Teams from MElanox and VMware based their work on current implementation of Neutron. Also this implementation was already tested in most deployment modes.</p>


<p class="">On this basis, I suggest:</p>
<ul class="">
<li class="">to merge <a href="https://review.openstack.org/#/c/103280">https://review.openstack.org/#/c/103280</a> in fuel-5.1</li>
<li class="">To base Neutron implementation for 6.0 get <a href="https://review.openstack.org/#/c/103280/"><span class="">https://review.openstack.org/#/c/103280/</span></a> and start working for adapt this for Juno Neutron.</li>


<li class="">Also we should discuss how to make HA-wrapper more pluggable.</li>
</ul></div></div>