yahoo-eng-team team mailing list archive

[LIU Yulong <yulong@xxxxxxxxxxxxx>]

Public bug reported:

HA/legacy router gateway secondary IPs was removed Unexpectedly.

First it process the external gateway:
https://github.com/openstack/neutron/blob/master/neutron/agent/l3/router_info.py#L1035
https://github.com/openstack/neutron/blob/master/neutron/agent/l3/router_info.py#L817

Then process the router floating IPs:
https://github.com/openstack/neutron/blob/master/neutron/agent/l3/router_info.py#L828

And then, the gateway secondary IPs has a /32 prefix, which will be treated as `fips_to_remove`:
https://github.com/openstack/neutron/blob/master/neutron/agent/l3/router_info.py#L290

Some related code:
existing_cidrs = self.get_router_cidrs(device), this existing_cidrs including the gateway secondary IPs:
https://github.com/openstack/neutron/blob/master/neutron/agent/l3/router_info.py#L304

gateway secondary IPs were removed here:
https://github.com/openstack/neutron/blob/master/neutron/agent/l3/router_info.py#L332


DVR/DVR_SNAT_HA router does not have this issue.

** Affects: neutron
     Importance: Undecided
     Assignee: LIU Yulong (dragon889)
         Status: New

** Summary changed:

- HA: ha router gateway secondary IPs was removed Unexpectedly
+ HA/legacy router gateway secondary IPs was removed unexpectedly

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1609647

Title:
  HA/legacy router gateway secondary IPs was removed unexpectedly

Status in neutron:
  New

Bug description:
  HA/legacy router gateway secondary IPs was removed Unexpectedly.

  First it process the external gateway:
  https://github.com/openstack/neutron/blob/master/neutron/agent/l3/router_info.py#L1035
  https://github.com/openstack/neutron/blob/master/neutron/agent/l3/router_info.py#L817

  Then process the router floating IPs:
  https://github.com/openstack/neutron/blob/master/neutron/agent/l3/router_info.py#L828

  And then, the gateway secondary IPs has a /32 prefix, which will be treated as `fips_to_remove`:
  https://github.com/openstack/neutron/blob/master/neutron/agent/l3/router_info.py#L290

  Some related code:
  existing_cidrs = self.get_router_cidrs(device), this existing_cidrs including the gateway secondary IPs:
  https://github.com/openstack/neutron/blob/master/neutron/agent/l3/router_info.py#L304

  gateway secondary IPs were removed here:
  https://github.com/openstack/neutron/blob/master/neutron/agent/l3/router_info.py#L332

  
  DVR/DVR_SNAT_HA router does not have this issue.

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1609647/+subscriptions