<div style="font-family: Helvetica; font-size: 14px;">I will spend some time on it during tonight/weekend to make sure<div>it’s removed and that we have reference implementation working at once.</div><div><br></div><div>I propose following the bug fix way, as it’s a tiny change.</div><div><br></div><div><a href="https://bugs.launchpad.net/neutron/+bug/1398768">https://bugs.launchpad.net/neutron/+bug/1398768</a></div><div><br></div><div>@amuller, I’m not sure I understand why does it need to be covered in the dhcp-agent side. Pushing extra routes to guest-vms?, I think we don’t cover the case of instances connected to an external network where we provide dhcp, but we may do that if we are or if we start covering that case anytime.</div><div><br></div></div>
<div><div><br></div><div><span style="font-size: 10pt;">Miguel Ángel Ajo</span></div><div><br></div></div>
<p style="color: #A0A0A8;">On Tuesday, 2 de December de 2014 at 15:48, Ihar Hrachyshka wrote:</p>
<blockquote type="cite" style="border-left-style:solid;border-width:1px;margin-left:0px;padding-left:10px;">
<span><div><div><div>-----BEGIN PGP SIGNED MESSAGE-----</div><div>Hash: SHA512</div><div><br></div><div>On 01/12/14 21:19, Kyle Mestery wrote:</div><blockquote type="cite"><div><div>On Mon, Dec 1, 2014 at 6:12 AM, Assaf Muller <<a href="mailto:amuller@redhat.com">amuller@redhat.com</a>></div><div>wrote:</div><blockquote type="cite"><div><div><br></div><div><br></div><div>----- Original Message -----</div><blockquote type="cite"><div><div><br></div><div>My proposal here, is, _let’s not deprecate this setting_, as</div><div>it’s a valid use case of a gateway configuration, and let’s</div><div>provide it on the reference implementation.</div></div></blockquote><div><br></div><div>I agree. As long as the reference implementation works with the</div><div>setting off there's no need to deprecate it. I still think the</div><div>default should be set to True though.</div><div><br></div><div>Keep in mind that the DHCP agent will need changes as well.</div></div></blockquote><div>++ to both suggestions Assaf. Thanks for bringing this up Miguel!</div></div></blockquote><div><br></div><div>Miguel, how about sending a patch that removes deprecation warning</div><div>from the help text then?</div><div><br></div><blockquote type="cite"><div><div><br></div><div>Kyle</div><div><br></div><blockquote type="cite"><blockquote type="cite"><div><div><br></div><div>TL;DR</div><div><br></div><div>I’ve been looking at this yesterday, during a test deployment </div><div>on a site where they provide external connectivity with the </div><div>gateway outside subnet.</div><div><br></div><div>And I needed to switch it of, to actually be able to have any</div><div>external connectivity.</div><div><br></div><div><a href="https://github.com/openstack/neutron/blob/master/etc/neutron.conf#L121">https://github.com/openstack/neutron/blob/master/etc/neutron.conf#L121</a></div></div></blockquote></blockquote></div></blockquote><div>This is handled by providing an on-link route to the gateway first,</div><blockquote type="cite"><blockquote type="cite"><div><blockquote type="cite"><div><div>and then adding the default gateway.</div><div><br></div><div>It looks to me very interesting (not only because it’s the only</div><div>way to work on that specific site [2][3][4]), because you can</div><div>dynamically wire RIPE blocks to your server, without needing to</div><div>use an specific IP for external routing or broadcast purposes,</div><div>and instead use the full block in openstack.</div><div><br></div><div><br></div><div>I have a tiny patch to support this on the neutron l3-agent [1]</div><div>I yet need to add the logic to check “gateway outside subnet”,</div><div>then add the “onlink” route.</div><div><br></div><div><br></div><div>[1]</div><div><br></div><div>diff --git a/neutron/agent/linux/interface.py </div><div>b/neutron/agent/linux/interface.py index 538527b..5a9f186</div><div>100644 --- a/neutron/agent/linux/interface.py +++</div><div>b/neutron/agent/linux/interface.py @@ -116,15 +116,16 @@ class</div><div>LinuxInterfaceDriver(object): namespace=namespace, ip=ip_cidr)</div><div><br></div><div>- if gateway: - device.route.add_gateway(gateway) - </div><div>new_onlink_routes = set(s['cidr'] for s in extra_subnets) + if</div><div>gateway: + new_onlink_routes.update([gateway]) </div><div>existing_onlink_routes =</div><div>set(device.route.list_onlink_routes()) for route in</div><div>new_onlink_routes - existing_onlink_routes: </div><div>device.route.add_onlink_route(route) for route in</div><div>existing_onlink_routes - new_onlink_routes: </div><div>device.route.delete_onlink_route(route) + if gateway: +</div><div>device.route.add_gateway(gateway)</div><div><br></div><div>def delete_conntrack_state(self, root_helper, namespace, ip): </div><div>"""Delete conntrack state associated with an IP address.</div><div><br></div><div>[2] <a href="http://www.soyoustart.com">http://www.soyoustart.com</a>/ [3] <a href="http://www.ovh.co.uk">http://www.ovh.co.uk</a>/ [4]</div><div><a href="http://www.kimsufi.com">http://www.kimsufi.com</a>/</div><div><br></div><div><br></div><div>Miguel Ángel Ajo</div><div><br></div><div><br></div><div><br></div><div><br></div><div>_______________________________________________ OpenStack-dev</div><div>mailing list <a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a> </div><div><a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a></div></div></blockquote><div><br></div><div><br></div><blockquote type="cite"><div></div></blockquote></div></blockquote></blockquote><div>_______________________________________________</div><blockquote type="cite"><div><blockquote type="cite"><div><div>OpenStack-dev mailing list <a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a> </div><div><a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a></div></div></blockquote><div><br></div><blockquote type="cite"><div></div></blockquote><div>_______________________________________________ OpenStack-dev</div><div>mailing list <a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a> </div><div><a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a></div></div></blockquote><div>-----BEGIN PGP SIGNATURE-----</div><div>Version: GnuPG/MacGPG2 v2.0.22 (Darwin)</div><div><br></div><div>iQEcBAEBCgAGBQJUfdEsAAoJEC5aWaUY1u57a4QIANjx/wOJJKlHJ1kiE5DQ80La</div><div>WP5DYwWj64MA+pDXPoE18+JZEV+7igHD7zeKb8pua4Ql+X/EDbLG5GK1ry4EV5RC</div><div>uKnO2tht/bLfrniirqoOcL5TqybW86ZP4TLtTzV1PdAQBNGoOaRU8pox5oAkZOmm</div><div>FrFVtBqoMtUAM9X8P7OHjkkvMLfoBinhWjlnyYWrzl6ZJtTCCipWJrVesHoWAL+F</div><div>DcWotMsSMkkCAolnDE1AST4Z6pRvj7Y4lhQyZGaOtDGkYoMPBb7PTaGIltzX3ijB</div><div>ZzDwz39o+kU9pY0/7Web6tFCEw+zFFr01rVBcQXDi5cJ2wRW7uT0J/9Aw0Rrn1M=</div><div>=coN8</div><div>-----END PGP SIGNATURE-----</div><div><br></div><div>_______________________________________________</div><div>OpenStack-dev mailing list</div><div><a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a></div><div><a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a></div></div></div></span>
</blockquote>
<div>
<br>
</div>