-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
After reboot(restart of l3 and l2 agents) of the node routers can be processed by l3 agent before openvswitch agent sets up appropriate ha ports. This change add notification for l3 agent that ha port becomes ACTIVE and keepalived can be enabled. note: Release notes added to specify l3 agent dependency on neutron server. Closes-bug: #1597461 Co-Authored-By: venkata anil <anilvenkata@redhat.com> (cherry picked from commit 25f5912) Conflicts: neutron/db/l3_hascheduler_db.py neutron/services/l3_router/l3_router_plugin.py neutron/tests/unit/plugins/ml2/test_plugin.py neutron/tests/functional/agent/l3/test_ha_router.py releasenotes/notes/l3ha-agent-server-dependency-1fcb775328ac4502.yaml Change-Id: Iedad1ccae45005efaaa74d5571df04197757d07a (cherry picked from commit 4ad841c) split out l3-ha specific test from TestMl2PortsV2 split out test_update_port_status_notify_port_event_after_update from ml2.test_plugin.TestMl2PortsV2 into TestMl2PortsV2WithL3 The change set of 25f5912 change id of Iedad1ccae45005efaaa74d5571df04197757d07a introduced a test, test_update_port_status_notify_port_event_after_update, that is valid only when l3 plugin support l3-ha. Such assumption isn't always true depending on actual ml2 driver. Since test cases in ml2.test_plugin is used as a common base for multiple drivers, test_update_port_status_notify_port_event_after_update, may or may not pass. So split out tests with very specific assumption into a new dedicated testcase so that each driver can safely reuse tests in tests/unit/plugin/ml2 based on their characteristics. Conflicts: neutron/tests/unit/plugins/ml2/test_plugin.py Closes-Bug: #1618601 Change-Id: Ie81dde976649111d029a7d107c99960aded64915 (cherry picked from commit 03c412f) Change-Id: Iedad1ccae45005efaaa74d5571df04197757d07a (cherry picked from commit 4ad841c)
- Loading branch information
1 parent
369faa2
commit 5860fb2
Showing
6 changed files
with
83 additions
and
1 deletion.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
6 changes: 6 additions & 0 deletions
6
releasenotes/notes/l3ha-agent-server-dependency-1fcb775328ac4502.yaml
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
--- | ||
upgrade: | ||
- Server notifies L3 HA agents when HA router interface | ||
port status becomes active. Then L3 HA agents spawn | ||
keepalived process. So, server has to be restarted | ||
before the L3 agents during upgrade. |