[L3] snat-ns will be initialized twice for DVR+HA routers during agent restart
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
neutron |
Fix Released
|
Medium
|
LIU Yulong | ||
neutron (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
Bionic |
Fix Released
|
Medium
|
Unassigned | ||
Focal |
Fix Released
|
Undecided
|
Unassigned | ||
Groovy |
Fix Released
|
Undecided
|
Unassigned | ||
Hirsute |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
If the DVR+HA router has external gateway, the snat-namespace will be initialized twice during agent restart.
And that initialized function will run many [1][2] external resource processing actions which will definitely increase the starting time of agent.
https:/
https:/
+++++++
SRU:
[Impact]
Longer l3-agent initialization time during restarts due to creation of snat namespace and setting corresponding sysctl twice.
With this fix, the initialization phase is triggered only once.
[Test Case]
* deploy Openstack on bionic queens (with neutron dvr l3 ha settings and debug mode on for neutron ) and create a router
(If stsstack-bundles are used, here are the commands
./generate-
./configure
# Configure creates a router with external gateway attached
)
* Restart neutron-l3-agent on one of the node
systemctl restart neutron-
* Check /var/log/
During initialization steps, following sysctl's are configured [1] [2].
Verify if the debug logs show sysctl execution statements are displayed twice after restart for snat namespace.
(If the fix is applied they should be displayed only once)
grep -inr snat-<router-id> /var/log/
Example log:
2718:2021-04-14 05:17:20.114 10868 DEBUG neutron.
[Where problems could occur]
no regression is expected, but if one occurs it would likely result in longer init time and/or failure to correctly init the snat-namespace
[1] https:/
[2] https:/
Changed in neutron: | |
assignee: | nobody → LIU Yulong (dragon889) |
tags: | added: l3-ha |
tags: | added: l3-dvr-backlog |
Changed in neutron (Ubuntu Bionic): | |
status: | New → Triaged |
importance: | Undecided → High |
importance: | High → Medium |
description: | updated |
Fix proposed to branch: master /review. opendev. org/692352
Review: https:/