----- Original Message ----- > > My proposal here, is, _let’s not deprecate this setting_, as it’s a valid use > case of a gateway configuration, and let’s provide it on the reference > implementation.
I agree. As long as the reference implementation works with the setting off there's no need to deprecate it. I still think the default should be set to True though. Keep in mind that the DHCP agent will need changes as well. > > TL;DR > > I’ve been looking at this yesterday, during a test deployment > on a site where they provide external connectivity with the > gateway outside subnet. > > And I needed to switch it of, to actually be able to have any external > connectivity. > > https://github.com/openstack/neutron/blob/master/etc/neutron.conf#L121 > > This is handled by providing an on-link route to the gateway first, > and then adding the default gateway. > > It looks to me very interesting (not only because it’s the only way to work > on that specific site [2][3][4]), because you can dynamically wire RIPE > blocks to your server, without needing to use an specific IP for external > routing or broadcast purposes, and instead use the full block in openstack. > > > I have a tiny patch to support this on the neutron l3-agent [1] I yet need to > add the logic to check “gateway outside subnet”, then add the “onlink” > route. > > > [1] > > diff --git a/neutron/agent/linux/interface.py > b/neutron/agent/linux/interface.py > index 538527b..5a9f186 100644 > --- a/neutron/agent/linux/interface.py > +++ b/neutron/agent/linux/interface.py > @@ -116,15 +116,16 @@ class LinuxInterfaceDriver(object): > namespace=namespace, > ip=ip_cidr) > > - if gateway: > - device.route.add_gateway(gateway) > - > new_onlink_routes = set(s['cidr'] for s in extra_subnets) > + if gateway: > + new_onlink_routes.update([gateway]) > existing_onlink_routes = set(device.route.list_onlink_routes()) > for route in new_onlink_routes - existing_onlink_routes: > device.route.add_onlink_route(route) > for route in existing_onlink_routes - new_onlink_routes: > device.route.delete_onlink_route(route) > + if gateway: > + device.route.add_gateway(gateway) > > def delete_conntrack_state(self, root_helper, namespace, ip): > """Delete conntrack state associated with an IP address. > > [2] http://www.soyoustart.com/ > [3] http://www.ovh.co.uk/ > [4] http://www.kimsufi.com/ > > > Miguel Ángel Ajo > > > > > _______________________________________________ > OpenStack-dev mailing list > OpenStack-dev@lists.openstack.org > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev > _______________________________________________ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
