Reviewed: https://review.openstack.org/264996
Committed:
https://git.openstack.org/cgit/openstack/neutron/commit/?id=30dab936e602cba7e35806e8a558b53eb8936f48
Submitter: Jenkins
Branch: master
commit 30dab936e602cba7e35806e8a558b53eb8936f48
Author: Kevin Benton <blak...@gmail.com>
Date: Thu Jan 7 14:28:24 2016 -0800
Only restrict gateway_ip change for router ports
The subnet update code was restricting gateway_ip changes if the
existing gateway IP belonged to a Neutron port. This was implemented
because changing the gateway will break all floating IP addresses if
the gateway is a Neutron router. However, this restriction makes it
possible to get a subnet stuck to an IP address that belongs to another
port (e.g. a compute port) so the user has to either delete the port
or change it's IP, both of which are disruptive.
This patch just changes the restriction so it only prevents gateway
IP changes if the current gateway IP belongs to a router. This
preserves the intent of the original change while allowing the subnet
to be updated off of IP addresses that belong to normal ports.
Change-Id: I4691505ef2fad6019e0d2fd80ff1b9e157662a29
Closes-bug: #1532004
** Changed in: neutron
Status: In Progress => Fix Released
--
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1532004
Title:
gateway update restriction should apply only to router interfaces
Status in neutron:
Fix Released
Bug description:
The restriction that prevents a subnet's gateway IP from being updated
if it points to an IP in use by a port can get the subnet in a stuck
state without messing with a port if the wrong gateway_ip is set.
Take the following example:
administrator@13:35:25:~/code/neutron$ neutron subnet-create bojangles
10.0.0.0/24 --name=bojangles --allocation-pool start=10.0.0.3,end=10.0.0.250
Created a new subnet:
+-------------------+--------------------------------------------+
| Field | Value |
+-------------------+--------------------------------------------+
| allocation_pools | {"start": "10.0.0.3", "end": "10.0.0.250"} |
| cidr | 10.0.0.0/24 |
| dns_nameservers | |
| enable_dhcp | True |
| gateway_ip | 10.0.0.1 |
| host_routes | |
| id | 21c9a4b3-a1d0-402f-8e1e-b463236cc612 |
| ip_version | 4 |
| ipv6_address_mode | |
| ipv6_ra_mode | |
| name | bojangles |
| network_id | 3c6ca69c-7662-441e-abc3-7a104aa603a1 |
| subnetpool_id | |
| tenant_id | de56db175c1d48b0bbe72f09a24a3b66 |
+-------------------+--------------------------------------------+
administrator@13:35:58:~/code/neutron$ neutron port-create bojangles
--fixed-ip ip_address=10.0.0.2
Created a new port:
+-----------------------+--------------------------------------------------------------------------------------------------+
| Field | Value
|
+-----------------------+--------------------------------------------------------------------------------------------------+
| admin_state_up | True
|
| allowed_address_pairs |
|
| binding:host_id |
|
| binding:profile | {}
|
| binding:vif_details | {}
|
| binding:vif_type | unbound
|
| binding:vnic_type | normal
|
| device_id |
|
| device_owner |
|
| dns_assignment | {"hostname": "host-10-0-0-2", "ip_address":
"10.0.0.2", "fqdn": "host-10-0-0-2.openstacklocal."} |
| dns_name |
|
| fixed_ips | {"subnet_id":
"21c9a4b3-a1d0-402f-8e1e-b463236cc612", "ip_address": "10.0.0.2"}
|
| id | 3cc9c4a0-1ea7-4353-a5e0-f112adf5fad8
|
| mac_address | fa:16:3e:09:d2:29
|
| name |
|
| network_id | 3c6ca69c-7662-441e-abc3-7a104aa603a1
|
| port_security_enabled | True
|
| security_groups | fa5cdb36-85e1-4182-bd08-d2ad22b5eb5b
|
| status | DOWN
|
| tenant_id | de56db175c1d48b0bbe72f09a24a3b66
|
+-----------------------+--------------------------------------------------------------------------------------------------+
administrator@13:36:05:~/code/neutron$ neutron subnet-update bojangles
--gateway_ip=10.0.0.2
Updated subnet: bojangles
administrator@13:36:14:~/code/neutron$ neutron subnet-update bojangles
--gateway_ip=10.0.0.1
Current gateway ip 10.0.0.2 already in use by port
3cc9c4a0-1ea7-4353-a5e0-f112adf5fad8. Unable to update.
The subnet is now pinned to the 10.0.0.2 port so the user will have to
destroy the port or update it's fixed IP before the subnet gateway can be
changed. If the port belongs to an active VM, this is disruptive.
The restriction should be loosened to only apply to router ports,
which is what it was originally intended for
(https://bugs.launchpad.net/neutron/+bug/1186322).
To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1532004/+subscriptions
--
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help : https://help.launchpad.net/ListHelp
