From 7b9af7888492bbca6037d0cdd1306e9f62600598 Mon Sep 17 00:00:00 2001 From: tomsou Date: Tue, 16 May 2017 08:50:49 +0000 Subject: Restore peering within testcase3 for Apex Change-Id: I7f3a4892e6ef73fb13512e427fab3e42ff1b69d9 Signed-off-by: tomsou --- docs/release/release-notes/release-notes.rst | 2 -- sdnvpn/test/functest/testcase_3.py | 29 +++++++++++++--------------- 2 files changed, 13 insertions(+), 18 deletions(-) diff --git a/docs/release/release-notes/release-notes.rst b/docs/release/release-notes/release-notes.rst index f5e04cf..08b236c 100644 --- a/docs/release/release-notes/release-notes.rst +++ b/docs/release/release-notes/release-notes.rst @@ -141,8 +141,6 @@ OpenStack NAT does not work. Instances created in subnets that are connected to the public network via a gateway should have external connectivity. This does not work and can be worked around by assigning a Floating IP to the instance [1]. -For Apex the peering with the virtual Peer is not working due to limitation -of apex networking. Workarounds ----------- diff --git a/sdnvpn/test/functest/testcase_3.py b/sdnvpn/test/functest/testcase_3.py index 85a8827..dbbdb59 100644 --- a/sdnvpn/test/functest/testcase_3.py +++ b/sdnvpn/test/functest/testcase_3.py @@ -262,26 +262,23 @@ def main(): results.add_failure(testcase) results.add_to_summary(0, "=") - # TODO remove this and include apex again - installer_type = str(os.environ['INSTALLER_TYPE'].lower()) - if installer_type != "apex": - - results.add_to_summary(0, '-') - results.add_to_summary(1, "Peer Quagga with OpenDaylight") - results.add_to_summary(0, '-') - - neighbor = quagga.odl_add_neighbor(fake_fip['fip_addr'], - controller_ext_ip, - controller) - peer = quagga.check_for_peering(controller) - if neighbor and peer: - results.add_success("Peering with quagga") - else: - results.add_failure("Peering with quagga") + results.add_to_summary(0, '-') + results.add_to_summary(1, "Peer Quagga with OpenDaylight") + results.add_to_summary(0, '-') + + neighbor = quagga.odl_add_neighbor(fake_fip['fip_addr'], + controller_ext_ip, + controller) + peer = quagga.check_for_peering(controller) finally: test_utils.detach_instance_from_ext_br(quagga_vm, compute) + if neighbor and peer: + results.add_success("Peering with quagga") + else: + results.add_failure("Peering with quagga") + test_utils.cleanup_nova(nova_client, floatingip_ids, instance_ids, image_ids) test_utils.cleanup_neutron(neutron_client, bgpvpn_ids, interfaces, -- cgit 1.2.3-korg