This commit completes the refactoring of the L3 agent callback mechanism.
The goal here is to also use the neutron/callbacks/ mechanism for L3 agent
notifications, instead of have two mechanisms.
[1] modified the L3 agent to send notifiactions for router create, udpate,
and delete events, using the neutron/callbacks/ mechanism.
[2] modified VPN to use this new mechanism, instead of the L3EventObservers
mechanism. Note:
[3] modified FW repo to no longer depended on the L3EventObserver and
related objects (it doesn't currently use the event notifications).
This commit removes the notifications for the L3EventObservers mechanism,
removed the related modules and tests, and adds in tests to verify that the
new notifications are called for the different events.
Once [1] and [2] are upstreamed, this commit can proceed.
Reviewed: https:/ /review. openstack. org/169876 /git.openstack. org/cgit/ openstack/ neutron/ commit/ ?id=9bca9ca84b7 6cc5bba03e9c0ff 42bceaf5d2b028
Committed: https:/
Submitter: Jenkins
Branch: master
commit 9bca9ca84b76cc5 bba03e9c0ff42bc eaf5d2b028
Author: Paul Michali <email address hidden>
Date: Wed Apr 1 13:47:43 2015 -0400
Refactoring cleanup for L3 agent callbacks
This commit completes the refactoring of the L3 agent callback mechanism.
The goal here is to also use the neutron/callbacks/ mechanism for L3 agent
notifications, instead of have two mechanisms.
[1] modified the L3 agent to send notifiactions for router create, udpate,
and delete events, using the neutron/callbacks/ mechanism.
[2] modified VPN to use this new mechanism, instead of the L3EventObservers
mechanism. Note:
[3] modified FW repo to no longer depended on the L3EventObserver and
related objects (it doesn't currently use the event notifications).
This commit removes the notifications for the L3EventObservers mechanism,
removed the related modules and tests, and adds in tests to verify that the
new notifications are called for the different events.
Once [1] and [2] are upstreamed, this commit can proceed.
Refs: /review. openstack. org/#/c/ 164466/ /review. openstack. org/#/c/ 165226/ /review. openstack. org/#/c/ 167275/
[1] https:/
[2] https:/
[3] https:/
Change-Id: I7c4b4ea5f9fb19 abb812665cdae5f b70c84fe3ec cc7cb5e59cdb7fb 95f61b13d47
Depends-On: If5040a827a6903
Closes-Bug: #1433552