This patch removes a conditional check in the update_router() method
which was verifying if snat was enabled in order to update the nat
rules. This check does not make sense in the update method as if snat
was disabled we should still call update_nat_rules() which will then
remove the NAT entry from the OVN NB DB.
Change-Id: Ice20d22365acaf33ee211b1e38b7d0bc151c1ba8
Closes-Bug: #1922089
Signed-off-by: Lucas Alvares Gomes <email address hidden>
(cherry picked from commit ddc8e625f714aebd65a454f23fedc6d9a9320e89)
Reviewed: https:/ /review. opendev. org/c/openstack /neutron/ +/792536 /opendev. org/openstack/ neutron/ commit/ 7e1df72c16f2e17 36b068387e76f5b 9070d6d5bf
Committed: https:/
Submitter: "Zuul (22348)"
Branch: stable/ussuri
commit 7e1df72c16f2e17 36b068387e76f5b 9070d6d5bf
Author: Lucas Alvares Gomes <email address hidden>
Date: Wed Apr 28 13:41:15 2021 +0100
[OVN] Fix: Disabling snat after it was enabled
This patch removes a conditional check in the update_router() method
which was verifying if snat was enabled in order to update the nat
rules. This check does not make sense in the update method as if snat
was disabled we should still call update_nat_rules() which will then
remove the NAT entry from the OVN NB DB.
Change-Id: Ice20d22365acaf 33ee211b1e38b7d 0bc151c1ba8 d65a454f23fedc6 d9a9320e89)
Closes-Bug: #1922089
Signed-off-by: Lucas Alvares Gomes <email address hidden>
(cherry picked from commit ddc8e625f714aeb