Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 1 | # vim: tabstop=4 shiftwidth=4 softtabstop=4 |
| 2 | |
ZhiQiang Fan | 39f9722 | 2013-09-20 04:49:44 +0800 | [diff] [blame] | 3 | # Copyright 2012 OpenStack Foundation |
Gavin Brebner | 0f465a3 | 2013-03-14 13:26:09 +0000 | [diff] [blame] | 4 | # Copyright 2013 Hewlett-Packard Development Company, L.P. |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 5 | # All Rights Reserved. |
| 6 | # |
| 7 | # Licensed under the Apache License, Version 2.0 (the "License"); you may |
| 8 | # not use this file except in compliance with the License. You may obtain |
| 9 | # a copy of the License at |
| 10 | # |
| 11 | # http://www.apache.org/licenses/LICENSE-2.0 |
| 12 | # |
| 13 | # Unless required by applicable law or agreed to in writing, software |
| 14 | # distributed under the License is distributed on an "AS IS" BASIS, WITHOUT |
| 15 | # WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the |
| 16 | # License for the specific language governing permissions and limitations |
| 17 | # under the License. |
| 18 | |
Attila Fazekas | aeeeefd | 2013-08-06 17:01:56 +0200 | [diff] [blame] | 19 | from tempest.common import debug |
Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 20 | from tempest.common.utils import data_utils |
Matthew Treinish | cb56994 | 2013-08-09 16:33:44 -0400 | [diff] [blame] | 21 | from tempest import config |
Brent Eagles | c26d452 | 2013-12-02 13:28:49 -0500 | [diff] [blame] | 22 | from tempest.openstack.common import jsonutils |
Matthew Treinish | 2b59f84 | 2013-09-09 20:32:51 +0000 | [diff] [blame] | 23 | from tempest.openstack.common import log as logging |
Sean Dague | 6dbc6da | 2013-05-08 17:49:46 -0400 | [diff] [blame] | 24 | from tempest.scenario import manager |
Brent Eagles | c26d452 | 2013-12-02 13:28:49 -0500 | [diff] [blame] | 25 | |
| 26 | import tempest.test |
Maru Newby | bd36022 | 2013-04-08 22:48:50 +0000 | [diff] [blame] | 27 | from tempest.test import attr |
Matthew Treinish | 2153ec0 | 2013-09-09 20:57:30 +0000 | [diff] [blame] | 28 | from tempest.test import services |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 29 | |
Sean Dague | 86bd842 | 2013-12-20 09:56:44 -0500 | [diff] [blame] | 30 | CONF = config.CONF |
Matthew Treinish | 2b59f84 | 2013-09-09 20:32:51 +0000 | [diff] [blame] | 31 | LOG = logging.getLogger(__name__) |
| 32 | |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 33 | |
Brent Eagles | c26d452 | 2013-12-02 13:28:49 -0500 | [diff] [blame] | 34 | class FloatingIPCheckTracker(object): |
| 35 | """ |
| 36 | Checking VM connectivity through floating IP addresses is bound to fail |
| 37 | if the floating IP has not actually been associated with the VM yet. |
| 38 | This helper class facilitates checking for floating IP assignments on |
| 39 | VMs. It only checks for a given IP address once. |
| 40 | """ |
| 41 | |
| 42 | def __init__(self, compute_client, floating_ip_map): |
| 43 | self.compute_client = compute_client |
Yair Fried | 9a551c4 | 2013-12-15 14:59:34 +0200 | [diff] [blame] | 44 | self.unchecked = floating_ip_map.copy() |
Brent Eagles | c26d452 | 2013-12-02 13:28:49 -0500 | [diff] [blame] | 45 | |
| 46 | def run_checks(self): |
| 47 | """Check for any remaining unverified floating IPs |
| 48 | |
| 49 | Gets VM details from nova and checks for floating IPs |
| 50 | within the returned information. Returns true when all |
| 51 | checks are complete and is suitable for use with |
| 52 | tempest.test.call_until_true() |
| 53 | """ |
| 54 | to_delete = [] |
| 55 | loggable_map = {} |
Yair Fried | 9a551c4 | 2013-12-15 14:59:34 +0200 | [diff] [blame] | 56 | for check_addr, server in self.unchecked.iteritems(): |
| 57 | serverdata = self.compute_client.servers.get(server.id) |
| 58 | ip_addr = [addr for sublist in serverdata.networks.values() for |
| 59 | addr in sublist] |
| 60 | if check_addr.floating_ip_address in ip_addr: |
| 61 | to_delete.append(check_addr) |
Brent Eagles | c26d452 | 2013-12-02 13:28:49 -0500 | [diff] [blame] | 62 | else: |
Yair Fried | 9a551c4 | 2013-12-15 14:59:34 +0200 | [diff] [blame] | 63 | loggable_map[server.id] = check_addr |
Brent Eagles | c26d452 | 2013-12-02 13:28:49 -0500 | [diff] [blame] | 64 | |
| 65 | for to_del in to_delete: |
| 66 | del self.unchecked[to_del] |
| 67 | |
| 68 | LOG.debug('Unchecked floating IPs: %s', |
| 69 | jsonutils.dumps(loggable_map)) |
| 70 | return len(self.unchecked) == 0 |
| 71 | |
| 72 | |
Sean Dague | 6dbc6da | 2013-05-08 17:49:46 -0400 | [diff] [blame] | 73 | class TestNetworkBasicOps(manager.NetworkScenarioTest): |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 74 | |
| 75 | """ |
| 76 | This smoke test suite assumes that Nova has been configured to |
Mark McClain | f2982e8 | 2013-07-06 17:48:03 -0400 | [diff] [blame] | 77 | boot VM's with Neutron-managed networking, and attempts to |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 78 | verify network connectivity as follows: |
| 79 | |
| 80 | * For a freshly-booted VM with an IP address ("port") on a given network: |
| 81 | |
Maru Newby | af292e8 | 2013-05-20 21:32:28 +0000 | [diff] [blame] | 82 | - the Tempest host can ping the IP address. This implies, but |
| 83 | does not guarantee (see the ssh check that follows), that the |
| 84 | VM has been assigned the correct IP address and has |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 85 | connectivity to the Tempest host. |
| 86 | |
Maru Newby | af292e8 | 2013-05-20 21:32:28 +0000 | [diff] [blame] | 87 | - the Tempest host can perform key-based authentication to an |
| 88 | ssh server hosted at the IP address. This check guarantees |
| 89 | that the IP address is associated with the target VM. |
| 90 | |
Yair Fried | 9a551c4 | 2013-12-15 14:59:34 +0200 | [diff] [blame] | 91 | - detach the floating-ip from the VM and verify that it becomes |
| 92 | unreachable |
| 93 | |
Yair Fried | 05db252 | 2013-11-18 11:02:10 +0200 | [diff] [blame] | 94 | - associate detached floating ip to a new VM and verify connectivity. |
| 95 | VMs are created with unique keypair so connectivity also asserts that |
| 96 | floating IP is associated with the new VM instead of the old one |
| 97 | |
Attila Fazekas | c3a095b | 2013-08-17 09:15:44 +0200 | [diff] [blame] | 98 | # TODO(mnewby) - Need to implement the following: |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 99 | - the Tempest host can ssh into the VM via the IP address and |
| 100 | successfully execute the following: |
| 101 | |
| 102 | - ping an external IP address, implying external connectivity. |
| 103 | |
| 104 | - ping an external hostname, implying that dns is correctly |
| 105 | configured. |
| 106 | |
| 107 | - ping an internal IP address, implying connectivity to another |
| 108 | VM on the same network. |
| 109 | |
| 110 | There are presumed to be two types of networks: tenant and |
| 111 | public. A tenant network may or may not be reachable from the |
| 112 | Tempest host. A public network is assumed to be reachable from |
| 113 | the Tempest host, and it should be possible to associate a public |
| 114 | ('floating') IP address with a tenant ('fixed') IP address to |
Chang Bo Guo | cc1623c | 2013-09-13 20:11:27 -0700 | [diff] [blame] | 115 | facilitate external connectivity to a potentially unroutable |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 116 | tenant IP address. |
| 117 | |
| 118 | This test suite can be configured to test network connectivity to |
| 119 | a VM via a tenant network, a public network, or both. If both |
| 120 | networking types are to be evaluated, tests that need to be |
| 121 | executed remotely on the VM (via ssh) will only be run against |
| 122 | one of the networks (to minimize test execution time). |
| 123 | |
| 124 | Determine which types of networks to test as follows: |
| 125 | |
| 126 | * Configure tenant network checks (via the |
| 127 | 'tenant_networks_reachable' key) if the Tempest host should |
| 128 | have direct connectivity to tenant networks. This is likely to |
| 129 | be the case if Tempest is running on the same host as a |
| 130 | single-node devstack installation with IP namespaces disabled. |
| 131 | |
| 132 | * Configure checks for a public network if a public network has |
| 133 | been configured prior to the test suite being run and if the |
| 134 | Tempest host should have connectivity to that public network. |
| 135 | Checking connectivity for a public network requires that a |
| 136 | value be provided for 'public_network_id'. A value can |
| 137 | optionally be provided for 'public_router_id' if tenants will |
| 138 | use a shared router to access a public network (as is likely to |
| 139 | be the case when IP namespaces are not enabled). If a value is |
| 140 | not provided for 'public_router_id', a router will be created |
| 141 | for each tenant and use the network identified by |
| 142 | 'public_network_id' as its gateway. |
| 143 | |
| 144 | """ |
| 145 | |
| 146 | @classmethod |
| 147 | def check_preconditions(cls): |
Gavin Brebner | 0f465a3 | 2013-03-14 13:26:09 +0000 | [diff] [blame] | 148 | super(TestNetworkBasicOps, cls).check_preconditions() |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 149 | cfg = cls.config.network |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 150 | if not (cfg.tenant_networks_reachable or cfg.public_network_id): |
| 151 | msg = ('Either tenant_networks_reachable must be "true", or ' |
| 152 | 'public_network_id must be defined.') |
Gavin Brebner | 0f465a3 | 2013-03-14 13:26:09 +0000 | [diff] [blame] | 153 | cls.enabled = False |
ivan-zhu | 1feeb38 | 2013-01-24 10:14:39 +0800 | [diff] [blame] | 154 | raise cls.skipException(msg) |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 155 | |
| 156 | @classmethod |
| 157 | def setUpClass(cls): |
| 158 | super(TestNetworkBasicOps, cls).setUpClass() |
| 159 | cls.check_preconditions() |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 160 | # TODO(mnewby) Consider looking up entities as needed instead |
| 161 | # of storing them as collections on the class. |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 162 | cls.security_groups = {} |
| 163 | cls.networks = [] |
Gavin Brebner | 851c350 | 2013-01-18 13:14:10 +0000 | [diff] [blame] | 164 | cls.subnets = [] |
| 165 | cls.routers = [] |
Yair Fried | 05db252 | 2013-11-18 11:02:10 +0200 | [diff] [blame] | 166 | cls.servers = {} |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 167 | cls.floating_ips = {} |
| 168 | |
Matthew Treinish | 2b59f84 | 2013-09-09 20:32:51 +0000 | [diff] [blame] | 169 | def _create_security_groups(self): |
Yair Fried | eb69f3f | 2013-10-10 13:18:16 +0300 | [diff] [blame] | 170 | self.security_groups[self.tenant_id] =\ |
| 171 | self._create_security_group_neutron(tenant_id=self.tenant_id) |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 172 | |
Matthew Treinish | 2b59f84 | 2013-09-09 20:32:51 +0000 | [diff] [blame] | 173 | def _check_networks(self): |
Attila Fazekas | c3a095b | 2013-08-17 09:15:44 +0200 | [diff] [blame] | 174 | # Checks that we see the newly created network/subnet/router via |
| 175 | # checking the result of list_[networks,routers,subnets] |
Gavin Brebner | 851c350 | 2013-01-18 13:14:10 +0000 | [diff] [blame] | 176 | seen_nets = self._list_networks() |
| 177 | seen_names = [n['name'] for n in seen_nets] |
| 178 | seen_ids = [n['id'] for n in seen_nets] |
| 179 | for mynet in self.networks: |
Gavin Brebner | 0f465a3 | 2013-03-14 13:26:09 +0000 | [diff] [blame] | 180 | self.assertIn(mynet.name, seen_names) |
| 181 | self.assertIn(mynet.id, seen_ids) |
Gavin Brebner | 851c350 | 2013-01-18 13:14:10 +0000 | [diff] [blame] | 182 | seen_subnets = self._list_subnets() |
| 183 | seen_net_ids = [n['network_id'] for n in seen_subnets] |
| 184 | seen_subnet_ids = [n['id'] for n in seen_subnets] |
| 185 | for mynet in self.networks: |
Gavin Brebner | 0f465a3 | 2013-03-14 13:26:09 +0000 | [diff] [blame] | 186 | self.assertIn(mynet.id, seen_net_ids) |
Gavin Brebner | 851c350 | 2013-01-18 13:14:10 +0000 | [diff] [blame] | 187 | for mysubnet in self.subnets: |
Gavin Brebner | 0f465a3 | 2013-03-14 13:26:09 +0000 | [diff] [blame] | 188 | self.assertIn(mysubnet.id, seen_subnet_ids) |
Gavin Brebner | 851c350 | 2013-01-18 13:14:10 +0000 | [diff] [blame] | 189 | seen_routers = self._list_routers() |
| 190 | seen_router_ids = [n['id'] for n in seen_routers] |
| 191 | seen_router_names = [n['name'] for n in seen_routers] |
| 192 | for myrouter in self.routers: |
Gavin Brebner | 0f465a3 | 2013-03-14 13:26:09 +0000 | [diff] [blame] | 193 | self.assertIn(myrouter.name, seen_router_names) |
| 194 | self.assertIn(myrouter.id, seen_router_ids) |
Gavin Brebner | 851c350 | 2013-01-18 13:14:10 +0000 | [diff] [blame] | 195 | |
Salvatore Orlando | 5ed3b6e | 2013-09-17 01:27:26 -0700 | [diff] [blame] | 196 | def _create_server(self, name, network): |
| 197 | tenant_id = network.tenant_id |
Yair Fried | 05db252 | 2013-11-18 11:02:10 +0200 | [diff] [blame] | 198 | keypair = self.create_keypair(name='keypair-%s' % name) |
Salvatore Orlando | 5ed3b6e | 2013-09-17 01:27:26 -0700 | [diff] [blame] | 199 | security_groups = [self.security_groups[tenant_id].name] |
| 200 | create_kwargs = { |
| 201 | 'nics': [ |
| 202 | {'net-id': network.id}, |
| 203 | ], |
Yair Fried | 05db252 | 2013-11-18 11:02:10 +0200 | [diff] [blame] | 204 | 'key_name': keypair.name, |
Salvatore Orlando | 5ed3b6e | 2013-09-17 01:27:26 -0700 | [diff] [blame] | 205 | 'security_groups': security_groups, |
| 206 | } |
Giulio Fidente | 61cadca | 2013-09-24 18:33:37 +0200 | [diff] [blame] | 207 | server = self.create_server(name=name, create_kwargs=create_kwargs) |
Yair Fried | 05db252 | 2013-11-18 11:02:10 +0200 | [diff] [blame] | 208 | self.servers[server] = keypair |
Salvatore Orlando | 5ed3b6e | 2013-09-17 01:27:26 -0700 | [diff] [blame] | 209 | return server |
| 210 | |
Matthew Treinish | 2b59f84 | 2013-09-09 20:32:51 +0000 | [diff] [blame] | 211 | def _create_servers(self): |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 212 | for i, network in enumerate(self.networks): |
Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 213 | name = data_utils.rand_name('server-smoke-%d-' % i) |
Yair Fried | 05db252 | 2013-11-18 11:02:10 +0200 | [diff] [blame] | 214 | self._create_server(name, network) |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 215 | |
Nachi Ueno | 4fb7ce6 | 2014-01-09 18:29:34 -0800 | [diff] [blame^] | 216 | def _log_console_output(self): |
| 217 | for server, key in self.servers.items(): |
| 218 | LOG.debug('Console output for %s', server.id) |
| 219 | LOG.debug(server.get_console_output()) |
| 220 | |
Matthew Treinish | 2b59f84 | 2013-09-09 20:32:51 +0000 | [diff] [blame] | 221 | def _check_tenant_network_connectivity(self): |
Sean Dague | 86bd842 | 2013-12-20 09:56:44 -0500 | [diff] [blame] | 222 | if not CONF.network.tenant_networks_reachable: |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 223 | msg = 'Tenant networks not configured to be reachable.' |
Matthew Treinish | 2b59f84 | 2013-09-09 20:32:51 +0000 | [diff] [blame] | 224 | LOG.info(msg) |
| 225 | return |
Maru Newby | af292e8 | 2013-05-20 21:32:28 +0000 | [diff] [blame] | 226 | # The target login is assumed to have been configured for |
| 227 | # key-based authentication by cloud-init. |
Sean Dague | 86bd842 | 2013-12-20 09:56:44 -0500 | [diff] [blame] | 228 | ssh_login = CONF.compute.image_ssh_user |
Brent Eagles | c26d452 | 2013-12-02 13:28:49 -0500 | [diff] [blame] | 229 | try: |
Yair Fried | 05db252 | 2013-11-18 11:02:10 +0200 | [diff] [blame] | 230 | for server, key in self.servers.items(): |
Brent Eagles | c26d452 | 2013-12-02 13:28:49 -0500 | [diff] [blame] | 231 | for net_name, ip_addresses in server.networks.iteritems(): |
| 232 | for ip_address in ip_addresses: |
| 233 | self._check_vm_connectivity(ip_address, ssh_login, |
Yair Fried | 05db252 | 2013-11-18 11:02:10 +0200 | [diff] [blame] | 234 | key.private_key) |
Yair Fried | a039f87 | 2014-01-02 12:11:10 +0200 | [diff] [blame] | 235 | except Exception: |
| 236 | LOG.exception('Tenant connectivity check failed') |
Nachi Ueno | 4fb7ce6 | 2014-01-09 18:29:34 -0800 | [diff] [blame^] | 237 | self._log_console_output() |
Brent Eagles | c26d452 | 2013-12-02 13:28:49 -0500 | [diff] [blame] | 238 | debug.log_ip_ns() |
Yair Fried | a039f87 | 2014-01-02 12:11:10 +0200 | [diff] [blame] | 239 | raise |
Brent Eagles | c26d452 | 2013-12-02 13:28:49 -0500 | [diff] [blame] | 240 | |
| 241 | def _wait_for_floating_ip_association(self): |
| 242 | ip_tracker = FloatingIPCheckTracker(self.compute_client, |
| 243 | self.floating_ips) |
| 244 | |
| 245 | self.assertTrue( |
| 246 | tempest.test.call_until_true( |
Sean Dague | 86bd842 | 2013-12-20 09:56:44 -0500 | [diff] [blame] | 247 | ip_tracker.run_checks, CONF.compute.build_timeout, |
| 248 | CONF.compute.build_interval), |
Brent Eagles | c26d452 | 2013-12-02 13:28:49 -0500 | [diff] [blame] | 249 | "Timed out while waiting for the floating IP assignments " |
| 250 | "to propagate") |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 251 | |
Yair Fried | 9a551c4 | 2013-12-15 14:59:34 +0200 | [diff] [blame] | 252 | def _create_and_associate_floating_ips(self): |
Sean Dague | 86bd842 | 2013-12-20 09:56:44 -0500 | [diff] [blame] | 253 | public_network_id = CONF.network.public_network_id |
Yair Fried | 05db252 | 2013-11-18 11:02:10 +0200 | [diff] [blame] | 254 | for server in self.servers.keys(): |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 255 | floating_ip = self._create_floating_ip(server, public_network_id) |
Yair Fried | 9a551c4 | 2013-12-15 14:59:34 +0200 | [diff] [blame] | 256 | self.floating_ips[floating_ip] = server |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 257 | |
Yair Fried | 9a551c4 | 2013-12-15 14:59:34 +0200 | [diff] [blame] | 258 | def _check_public_network_connectivity(self, should_connect=True): |
Maru Newby | af292e8 | 2013-05-20 21:32:28 +0000 | [diff] [blame] | 259 | # The target login is assumed to have been configured for |
| 260 | # key-based authentication by cloud-init. |
Sean Dague | 86bd842 | 2013-12-20 09:56:44 -0500 | [diff] [blame] | 261 | ssh_login = CONF.compute.image_ssh_user |
Nachi Ueno | 4fb7ce6 | 2014-01-09 18:29:34 -0800 | [diff] [blame^] | 262 | LOG.debug('checking network connections') |
Attila Fazekas | aeeeefd | 2013-08-06 17:01:56 +0200 | [diff] [blame] | 263 | try: |
Yair Fried | 9a551c4 | 2013-12-15 14:59:34 +0200 | [diff] [blame] | 264 | for floating_ip, server in self.floating_ips.iteritems(): |
| 265 | ip_address = floating_ip.floating_ip_address |
Yair Fried | 05db252 | 2013-11-18 11:02:10 +0200 | [diff] [blame] | 266 | private_key = None |
| 267 | if should_connect: |
| 268 | private_key = self.servers[server].private_key |
Yair Fried | 9a551c4 | 2013-12-15 14:59:34 +0200 | [diff] [blame] | 269 | self._check_vm_connectivity(ip_address, |
| 270 | ssh_login, |
| 271 | private_key, |
| 272 | should_connect=should_connect) |
Yair Fried | a039f87 | 2014-01-02 12:11:10 +0200 | [diff] [blame] | 273 | except Exception: |
| 274 | LOG.exception('Public network connectivity check failed') |
Nachi Ueno | 4fb7ce6 | 2014-01-09 18:29:34 -0800 | [diff] [blame^] | 275 | self._log_console_output() |
Attila Fazekas | aeeeefd | 2013-08-06 17:01:56 +0200 | [diff] [blame] | 276 | debug.log_ip_ns() |
Yair Fried | a039f87 | 2014-01-02 12:11:10 +0200 | [diff] [blame] | 277 | raise |
Matthew Treinish | 2b59f84 | 2013-09-09 20:32:51 +0000 | [diff] [blame] | 278 | |
Yair Fried | 9a551c4 | 2013-12-15 14:59:34 +0200 | [diff] [blame] | 279 | def _disassociate_floating_ips(self): |
| 280 | for floating_ip, server in self.floating_ips.iteritems(): |
| 281 | self._disassociate_floating_ip(floating_ip) |
| 282 | self.floating_ips[floating_ip] = None |
| 283 | |
Yair Fried | 05db252 | 2013-11-18 11:02:10 +0200 | [diff] [blame] | 284 | def _reassociate_floating_ips(self): |
| 285 | network = self.networks[0] |
| 286 | for floating_ip in self.floating_ips.keys(): |
| 287 | name = data_utils.rand_name('new_server-smoke-') |
| 288 | # create a new server for the floating ip |
| 289 | server = self._create_server(name, network) |
| 290 | self._associate_floating_ip(floating_ip, server) |
| 291 | self.floating_ips[floating_ip] = server |
| 292 | |
Matthew Treinish | 2b59f84 | 2013-09-09 20:32:51 +0000 | [diff] [blame] | 293 | @attr(type='smoke') |
Matthew Treinish | 2153ec0 | 2013-09-09 20:57:30 +0000 | [diff] [blame] | 294 | @services('compute', 'network') |
Matthew Treinish | 2b59f84 | 2013-09-09 20:32:51 +0000 | [diff] [blame] | 295 | def test_network_basic_ops(self): |
Matthew Treinish | 2b59f84 | 2013-09-09 20:32:51 +0000 | [diff] [blame] | 296 | self._create_security_groups() |
| 297 | self._create_networks() |
| 298 | self._check_networks() |
| 299 | self._create_servers() |
Yair Fried | 9a551c4 | 2013-12-15 14:59:34 +0200 | [diff] [blame] | 300 | self._create_and_associate_floating_ips() |
Brent Eagles | c26d452 | 2013-12-02 13:28:49 -0500 | [diff] [blame] | 301 | self._wait_for_floating_ip_association() |
Salvatore Orlando | 8d076cb | 2013-09-16 11:00:44 -0700 | [diff] [blame] | 302 | self._check_tenant_network_connectivity() |
Yair Fried | 9a551c4 | 2013-12-15 14:59:34 +0200 | [diff] [blame] | 303 | self._check_public_network_connectivity(should_connect=True) |
| 304 | self._disassociate_floating_ips() |
| 305 | self._check_public_network_connectivity(should_connect=False) |
Yair Fried | 05db252 | 2013-11-18 11:02:10 +0200 | [diff] [blame] | 306 | self._reassociate_floating_ips() |
| 307 | self._wait_for_floating_ip_association() |
| 308 | self._check_public_network_connectivity(should_connect=True) |