subnetpool address scope change event should include the old address scope id

Bug #1598735 reported by Steve Ruan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Won't Fix
Wishlist
Unassigned

Bug Description

in neutron.db.db_base_plugin_v2,
def update_subnetpool(self, context, id, subnetpool):
....
            if address_scope_changed:
            # Notify about the update of subnetpool's address scope
            kwargs = {'context': context, 'subnetpool_id': id}
            registry.notify(resources.SUBNETPOOL_ADDRESS_SCOPE,
                            events.AFTER_UPDATE,
                            self.update_subnetpool,
                            **kwargs)

This kwargs ONLY includes subnetpool_id, in some cases, we want to know the subnetpool old address scope id.

Here is the use case,
To develop bgp vpn in neutron-dynamic-routing, each bgpvpn is associated to address scope, bgpvpn routes only include the subnets of same address scope.
If the subnetpool change its address scope, for some bgpvpnvs associated to old address scope, it should delete the routes; for some bgpvpns associated to new address scope, it should add new routes.

If this event does not include old address_scope_id, bgpvpn can not delete old routes.

Changed in neutron:
importance: Undecided → Wishlist
Steve Ruan (ruansx)
Changed in neutron:
assignee: nobody → Steve Ruan (ruansx)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

Fix proposed to branch: master
Review: https://review.openstack.org/337662

Changed in neutron:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on neutron (master)

Change abandoned by Armando Migliaccio (<email address hidden>) on branch: master
Review: https://review.openstack.org/337662
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Changed in neutron:
status: In Progress → Confirmed
assignee: Steve Ruan (ruansx) → nobody
tags: added: low-hanging-fruit
zainub (zainub.wahid)
Changed in neutron:
assignee: nobody → zainub (zainub.wahid)
zainub (zainub.wahid)
Changed in neutron:
assignee: zainub (zainub.wahid) → nobody
zainub (zainub.wahid)
Changed in neutron:
assignee: nobody → zainub (zainub.wahid)
assignee: zainub (zainub.wahid) → nobody
Changed in neutron:
assignee: nobody → Prince Nana Owusu Boateng (nanaboat)
Changed in neutron:
assignee: Prince Nana Owusu Boateng (nanaboat) → nobody
Changed in neutron:
status: Confirmed → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.