commit 49e6e141f254762f63ba8347ead84e28eefca838
Author: Daniel Alvarez <email address hidden>
Date: Mon Nov 12 13:21:41 2018 +0100
Fix Tempest and Unit tests
This patch is a squash of two commits that fix both tempest and
unit tests:
1. Port Groups: Fix bug when updating a port to trusted
When a normal port with port security enabled (ie. belonging to
default drop Port Group) gets updated to a trusted port (ie.
router port), the existing code won't remove the port from the
Port Group so all traffic gets dropped.
This patch fixes the issue by checking this condition in the
update_port method of the ovn_client module.
2. UT: Fix unit tests failures
After 3316b45665a99b0f61e45a8c7facf538618861bf got merged in
Neutron, our gate is blocked due to unit tests failing.
After some debugging, it looks like no longer importing
neutron.db.api is the culprit. This patch is importing it
from our unit tests to unbreak the gate while the actual
root case is determined.
Change-Id: I3d30ec18f0c9df256c8f5846b52ab619835b5e32
Closes-Bug: #1802373
Partial-Bug: #1802369
Signed-off-by: Daniel Alvarez <email address hidden>
(cherry picked from commit 605ad27f053a1ef9ffb3601c6023987ddb6a22e1)
Reviewed: https:/ /review. opendev. org/681568 /git.openstack. org/cgit/ openstack/ networking- ovn/commit/ ?id=49e6e141f25 4762f63ba8347ea d84e28eefca838
Committed: https:/
Submitter: Zuul
Branch: stable/queens
commit 49e6e141f254762 f63ba8347ead84e 28eefca838
Author: Daniel Alvarez <email address hidden>
Date: Mon Nov 12 13:21:41 2018 +0100
Fix Tempest and Unit tests
This patch is a squash of two commits that fix both tempest and
unit tests:
1. Port Groups: Fix bug when updating a port to trusted
When a normal port with port security enabled (ie. belonging to
default drop Port Group) gets updated to a trusted port (ie.
router port), the existing code won't remove the port from the
Port Group so all traffic gets dropped.
This patch fixes the issue by checking this condition in the
update_port method of the ovn_client module.
2. UT: Fix unit tests failures
After 3316b45665a99b0 f61e45a8c7facf5 38618861bf got merged in
Neutron, our gate is blocked due to unit tests failing.
After some debugging, it looks like no longer importing
neutron.db.api is the culprit. This patch is importing it
from our unit tests to unbreak the gate while the actual
root case is determined.
Change-Id: I3d30ec18f0c9df 256c8f5846b52ab 619835b5e32 9ffb3601c602398 7ddb6a22e1)
Closes-Bug: #1802373
Partial-Bug: #1802369
Signed-off-by: Daniel Alvarez <email address hidden>
(cherry picked from commit 605ad27f053a1ef