<div dir="ltr"><div>I'm only part way through reviewing this, but I think there's a fundamental error in it.  We were at one point going to use 'enable_dhcp' in the current set of flags to indicate something meaningful, but eventually we decided that its current behaviour (despite the naming) really meant 'no address assignment protocols are active' - which is why setting enable_dhcp=False completely disables DHCP and RA activity in Neutron.  (I believe Mark pointed that out, and agitated for backward compatibility, though I couldn't tell you which forum it was in.)  Your proposal reverses that decision, and says that it can be set to false and yet RAs will still be sent out.<br>

<br>It's also why there are two additional attributes, because a single option isn't really 
expressive enough once you consider the provider network cases.  You are, in your own way, also using two attributes, 
by repurposing enable_dhcp - your attribute values are different, but you're
 doing it for similar reasons.<br><br></div><div>I'll put up my other review comments a bit later on when I've had another think.<br></div><div>-- <br></div>Ian.<br><br></div><div class="gmail_extra"><br><br><div class="gmail_quote">

On 11 June 2014 06:07, Robert Li (baoli) <span dir="ltr"><<a href="mailto:baoli@cisco.com" target="_blank">baoli@cisco.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">





<div style="word-wrap:break-word;color:rgb(0,0,0);font-size:14px;font-family:Calibri,sans-serif">
<div>Hi,</div>
<div><br>
</div>
<div>I was mistakenly reusing the Blueprint <a href="https://blueprints.launchpad.net/neutron/+spec/neutron-ipv6-radvd-ra" target="_blank">
https://blueprints.launchpad.net/neutron/+spec/neutron-ipv6-radvd-ra</a> to draft up the ipv6 RA support in neutron. I apologize for any confusion that this may have caused. To correct it, I created a new blueprint
<a href="https://blueprints.launchpad.net/neutron/+spec/neutron-ipv6-ra" target="_blank">https://blueprints.launchpad.net/neutron/+spec/neutron-ipv6-ra</a> and linked it to a neutron spec <a rel="nofollow" href="https://review.openstack.org/92164" style="max-width:60em;color:rgb(0,51,170);text-decoration:none;font-family:sans-serif;font-size:12px;line-height:18px;background-color:rgb(255,255,255)" target="_blank">https:/<u></u>/review.<u></u>openstack.<u></u>org/92164</a>. </div>


<div><br>
</div>
<div>The basic idea behind this blueprint is that neutron dhcp service as it is can hand out IPv6 addresses either in IPv6 only data network or in a dual stack data network. But the RA service including SLAAC support is missing in neutron. This service is important.
 Without RA, VMs won’t be able to automatically install default routes. Without SLAAC, a deployment that prefers SLAAC won’t be able to do it with neutron. The BP takes a straightforward approach to achieve that without requiring significant change to the existing
 dhcp service.</div>
<div><br>
</div>
<div>Any feedback is appreciated. </div>
<div><br>
</div>
<div>thanks,</div>
<div>Robert</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>