Hi Xu Han & Sean,

Is this code going to be committed as it is? Based on this morning's
discussion, I thought that the IP address used to install the RA rule
comes from the qr-xxx interface's LLA address. I think that I'm confused.

Also this bug: Allow LLA as router interface of IPv6 subnet
https://review.openstack.org/76125 was created due to comments to 72252.
If We don't need to create a new LLA for the gateway IP, is the fix still
needed? 

Just trying to sync up with you guys on them.

Thanks,
Robert



On 3/4/14 3:02 AM, "Sean M. Collins (Code Review)" <[email protected]>
wrote:

>Sean M. Collins has posted comments on this change.
>
>Change subject: Permit ICMPv6 RAs only from known routers
>......................................................................
>
>
>Patch Set 4: Looks good to me, but someone else must approve
>
>Automatically re-added by Gerrit trivial rebase detection script.
>
>--
>To view, visit https://review.openstack.org/72252
>To unsubscribe, visit https://review.openstack.org/settings
>
>Gerrit-MessageType: comment
>Gerrit-Change-Id: I1d5c7aaa8e4cf057204eb746c0faab2c70409a94
>Gerrit-PatchSet: 4
>Gerrit-Project: openstack/neutron
>Gerrit-Branch: master
>Gerrit-Owner: Xu Han Peng <[email protected]>
>Gerrit-Reviewer: Arista Testing <[email protected]>
>Gerrit-Reviewer: Baodong (Robert) Li <[email protected]>
>Gerrit-Reviewer: Big Switch CI <[email protected]>
>Gerrit-Reviewer: Brian Haley <[email protected]>
>Gerrit-Reviewer: Brocade CI <[email protected]>
>Gerrit-Reviewer: Cisco Neutron CI <[email protected]>
>Gerrit-Reviewer: Hyper-V CI <[email protected]>
>Gerrit-Reviewer: Jenkins
>Gerrit-Reviewer: Midokura CI Bot <[email protected]>
>Gerrit-Reviewer: Miguel Angel Ajo <[email protected]>
>Gerrit-Reviewer: NEC OpenStack CI <[email protected]>
>Gerrit-Reviewer: Neutron Ryu <[email protected]>
>Gerrit-Reviewer: Nuage CI <[email protected]>
>Gerrit-Reviewer: One Convergence CI <[email protected]>
>Gerrit-Reviewer: PLUMgrid CI <[email protected]>
>Gerrit-Reviewer: Sean M. Collins <[email protected]>
>Gerrit-Reviewer: Xu Han Peng <[email protected]>
>Gerrit-Reviewer: mark mcclain <[email protected]>


_______________________________________________
OpenStack-dev mailing list
[email protected]
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Reply via email to