<div dir="ltr">Hi Kyle,<div><br></div><div>I think conceptually your approach is fine.</div><div>I would have had concerns if you were trying to manage ODL life cycle through devstack (like installing/uninstalling it or configuring the ODL controller).</div>
<div>But looking at your code it seems you're just setting up the host so that it could work with opendaylight.</div><div><br></div><div>I agree however that extras.d is probably not the right place, as devstack already has hooks in places for plugin configuration.</div>
<div>I think they are at least:</div><div>- configure</div><div>- check</div><div>- init</div><div>- install</div><div>- start </div><div><br></div><div>big switch, midokura, nec, ryu, and nsx already use these hooks.</div>
<div>I appreciate the fact that since this is a mech driver rather than a plugin, this solution won't work out of the box, but at first glance it should not be to hard to adapt it.</div><div><br></div><div>Salvatore</div>
<div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 26 February 2014 22:47, Kyle Mestery <span dir="ltr"><<a href="mailto:mestery@noironetworks.com" target="_blank">mestery@noironetworks.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">So, I have this review [1] which attempts to add support for OpenDaylight<div>to devstack. What this currently does, in Patch 7, is that it uses the</div>
<div>"extras" functionality of devstack to program the OVS on the host so that</div>
<div>OpenDaylight can control it. On teardown, it does the reverse. Simple and</div><div>straightforward. I've received feedback this isn't the correct approach here,</div><div>and that using a plugin approach in lib/neutron_plugin/opendaylight would</div>

<div>be better. I need hooks for when devstack is finished running, and when</div><div>unstack is called. Those don't appear in the plugin interface for Neutron</div><div>in devstack.</div><div><br></div><div>Another point of inconsistency I'd like to bring up is the fact that patches</div>

<div>for Neutron in devstack which propose running an Open Source controller</div><div>are being flagged with -1. However, the Ryu plugin is already doing this. I</div><div>suspect it was grandfathered in, but it sets an inconsistent precedent here.</div>

<div>I propose we either remove Ryu from devstack, or continue to let other</div><div>Open Source SDN controllers run inside devstack. Please see Patch 6</div><div>of the review below for the minimal work it took me to add OpenDaylight</div>

<div>there.</div><div><br></div><div>Feedback appreciated here, I've been sitting on this devstack patch with</div><div>minimal changes for a month. I'm also working with the Linux Foundation</div><div>for the 3rd party testing requirements for ODL so the ML2 MechanismDriver</div>

<div>can also go in.</div><div><br></div><div>Thanks,</div><div>Kyle</div><div><br></div><div>[1] <a href="https://review.openstack.org/#/c/69774/" target="_blank">https://review.openstack.org/#/c/69774/</a></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>