Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Ensure traffic is not centralized if DVR is enabled
There is no need to clear the external_mac if DVR is enabled, not even when the port is down. This patch ensures the external_mac is only deleted when DVR is not enabled. Without this patch, if a VM with a floating IP gets deleted, and DVR is enabled, during some time the traffic gets (wrongly) centralized while it should not. And it is also generating more load on the OVN side unnecesarily. Closes-Bug: #2025264 Change-Id: I89db15dd1b629bc963f3b63926391a4a02cbedf7 (cherry picked from commit 0090572)
- Loading branch information
Showing
3 changed files
with
28 additions
and
15 deletions.
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
--- | ||
other: | ||
- | | ||
The external_mac entry in the NAT table is used to distribute/centralize | ||
the traffic to the FIPs. When there is an external_mac set the traffic | ||
is distributed (DVR). When it is empty it is centralized through the | ||
gateway port (no DVR). Upon port status transition to down, the | ||
external_mac was removed regardless of DVR being enabled or not, leading | ||
to centralize the FIP traffic for DVR -- though it was for down ports that | ||
won't accept traffic anyway. |