commit 3aa6534262d6ea20ce3234dd41ab120c5ae035ef
author Xavier León <xavi.leon@midokura.com> Tue May 12 15:24:28 2015 +0000
committer Xavi León <xavi.leon@midokura.com> Fri May 29 12:47:47 2015 +0200
tree 2104295a2695e326caaa6cac3c5634b2cf377da1
parent 4a5fce2deb70ca85c70c3ec0d05ec1528028461d

Do not consider subnets of shared public nets

Test test_external_network_visibility in
tempest.api.network.test_network fails when running
tempest against a deployment with public networks which
are shared among non-admin tenants.

This patch skips the public networks whose shared
property is set to true, as their subnets will be visible to
non-admin tenants.

Change-Id: Ieafb8e4094e603e259d68c4f485194a602dfc0cf
Closes-Bug: #1425962

tempest/api/network/test_networks.py

diff --git a/tempest/api/network/test_networks.py b/tempest/api/network/test_networks.py
index 5e3e374..7a96f34 100644
--- a/tempest/api/network/test_networks.py
+++ b/tempest/api/network/test_networks.py
@@ -393,7 +393,9 @@
                                       " in filtered list (%s)." % nonexternal)
         self.assertIn(CONF.network.public_network_id, networks)
 
-        subnets_iter = (network['subnets'] for network in body['networks'])
+        subnets_iter = (network['subnets']
+                        for network in body['networks']
+                        if not network['shared'])
         # subnets_iter is a list (iterator) of lists. This flattens it to a
         # list of UUIDs
         public_subnets_iter = itertools.chain(*subnets_iter)