ZhiQiang Fan | 39f9722 | 2013-09-20 04:49:44 +0800 | [diff] [blame] | 1 | # Copyright 2012 OpenStack Foundation |
Gavin Brebner | 0f465a3 | 2013-03-14 13:26:09 +0000 | [diff] [blame] | 2 | # Copyright 2013 Hewlett-Packard Development Company, L.P. |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 3 | # All Rights Reserved. |
| 4 | # |
| 5 | # Licensed under the Apache License, Version 2.0 (the "License"); you may |
| 6 | # not use this file except in compliance with the License. You may obtain |
| 7 | # a copy of the License at |
| 8 | # |
| 9 | # http://www.apache.org/licenses/LICENSE-2.0 |
| 10 | # |
| 11 | # Unless required by applicable law or agreed to in writing, software |
| 12 | # distributed under the License is distributed on an "AS IS" BASIS, WITHOUT |
| 13 | # WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the |
| 14 | # License for the specific language governing permissions and limitations |
| 15 | # under the License. |
Yair Fried | 2d2f3fe | 2014-02-24 16:19:20 +0200 | [diff] [blame] | 16 | import collections |
Yair Fried | 3097dc1 | 2014-01-26 08:46:43 +0200 | [diff] [blame] | 17 | import re |
Matthew Treinish | 96e9e88 | 2014-06-09 18:37:19 -0400 | [diff] [blame] | 18 | |
Doug Hellmann | 583ce2c | 2015-03-11 14:55:46 +0000 | [diff] [blame] | 19 | from oslo_log import log as logging |
Adam Gandelman | 7186f7a | 2014-07-23 09:28:56 -0400 | [diff] [blame] | 20 | import testtools |
Yair Fried | 3097dc1 | 2014-01-26 08:46:43 +0200 | [diff] [blame] | 21 | |
Andrea Frittoli | cd36841 | 2017-08-14 21:37:56 +0100 | [diff] [blame] | 22 | from tempest.common import utils |
Ken'ichi Ohmichi | e91a0c6 | 2015-08-13 02:09:16 +0000 | [diff] [blame] | 23 | from tempest.common import waiters |
Matthew Treinish | cb56994 | 2013-08-09 16:33:44 -0400 | [diff] [blame] | 24 | from tempest import config |
Jordan Pittier | 9e227c5 | 2016-02-09 14:35:18 +0100 | [diff] [blame] | 25 | from tempest.lib.common.utils import test_utils |
Sean Dague | 17487fb | 2016-08-08 10:44:20 -0400 | [diff] [blame] | 26 | from tempest.lib import decorators |
guo yunxian | ebb15f2 | 2016-11-01 21:03:35 +0800 | [diff] [blame] | 27 | from tempest.lib import exceptions |
Sean Dague | 6dbc6da | 2013-05-08 17:49:46 -0400 | [diff] [blame] | 28 | from tempest.scenario import manager |
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 | |
Yair Fried | 2d2f3fe | 2014-02-24 16:19:20 +0200 | [diff] [blame] | 33 | Floating_IP_tuple = collections.namedtuple('Floating_IP_tuple', |
| 34 | ['floating_ip', 'server']) |
| 35 | |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 36 | |
Andrea Frittoli | 4971fc8 | 2014-09-25 10:22:20 +0100 | [diff] [blame] | 37 | class TestNetworkBasicOps(manager.NetworkScenarioTest): |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 38 | |
Ken'ichi Ohmichi | 2e2ee19 | 2015-11-19 09:48:27 +0000 | [diff] [blame] | 39 | """The test suite of network basic operations |
| 40 | |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 41 | This smoke test suite assumes that Nova has been configured to |
Mark McClain | f2982e8 | 2013-07-06 17:48:03 -0400 | [diff] [blame] | 42 | boot VM's with Neutron-managed networking, and attempts to |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 43 | verify network connectivity as follows: |
| 44 | |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 45 | There are presumed to be two types of networks: tenant and |
| 46 | public. A tenant network may or may not be reachable from the |
| 47 | Tempest host. A public network is assumed to be reachable from |
| 48 | the Tempest host, and it should be possible to associate a public |
| 49 | ('floating') IP address with a tenant ('fixed') IP address to |
Chang Bo Guo | cc1623c | 2013-09-13 20:11:27 -0700 | [diff] [blame] | 50 | facilitate external connectivity to a potentially unroutable |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 51 | tenant IP address. |
| 52 | |
| 53 | This test suite can be configured to test network connectivity to |
| 54 | a VM via a tenant network, a public network, or both. If both |
| 55 | networking types are to be evaluated, tests that need to be |
| 56 | executed remotely on the VM (via ssh) will only be run against |
| 57 | one of the networks (to minimize test execution time). |
| 58 | |
| 59 | Determine which types of networks to test as follows: |
| 60 | |
| 61 | * Configure tenant network checks (via the |
Sean Dague | ed6e586 | 2016-04-04 10:49:13 -0400 | [diff] [blame] | 62 | 'project_networks_reachable' key) if the Tempest host should |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 63 | have direct connectivity to tenant networks. This is likely to |
| 64 | be the case if Tempest is running on the same host as a |
| 65 | single-node devstack installation with IP namespaces disabled. |
| 66 | |
| 67 | * Configure checks for a public network if a public network has |
| 68 | been configured prior to the test suite being run and if the |
| 69 | Tempest host should have connectivity to that public network. |
| 70 | Checking connectivity for a public network requires that a |
| 71 | value be provided for 'public_network_id'. A value can |
| 72 | optionally be provided for 'public_router_id' if tenants will |
| 73 | use a shared router to access a public network (as is likely to |
| 74 | be the case when IP namespaces are not enabled). If a value is |
| 75 | not provided for 'public_router_id', a router will be created |
| 76 | for each tenant and use the network identified by |
| 77 | 'public_network_id' as its gateway. |
| 78 | |
| 79 | """ |
| 80 | |
| 81 | @classmethod |
Emily Hugenbruch | 5e2d2a2 | 2015-02-25 21:35:45 +0000 | [diff] [blame] | 82 | def skip_checks(cls): |
| 83 | super(TestNetworkBasicOps, cls).skip_checks() |
Federico Ressi | 2d6bcaa | 2018-04-11 12:37:36 +0200 | [diff] [blame] | 84 | if not (CONF.network.project_networks_reachable or |
| 85 | CONF.network.public_network_id): |
Sean Dague | ed6e586 | 2016-04-04 10:49:13 -0400 | [diff] [blame] | 86 | msg = ('Either project_networks_reachable must be "true", or ' |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 87 | 'public_network_id must be defined.') |
ivan-zhu | 1feeb38 | 2013-01-24 10:14:39 +0800 | [diff] [blame] | 88 | raise cls.skipException(msg) |
Yoshihiro Kaneko | 0567026 | 2014-01-18 19:22:44 +0900 | [diff] [blame] | 89 | for ext in ['router', 'security-group']: |
Andrea Frittoli | cd36841 | 2017-08-14 21:37:56 +0100 | [diff] [blame] | 90 | if not utils.is_extension_enabled(ext, 'network'): |
Yoshihiro Kaneko | 0567026 | 2014-01-18 19:22:44 +0900 | [diff] [blame] | 91 | msg = "%s extension not enabled." % ext |
| 92 | raise cls.skipException(msg) |
Matthew Treinish | 3312de3 | 2017-05-19 12:08:17 -0400 | [diff] [blame] | 93 | if not CONF.network_feature_enabled.floating_ips: |
| 94 | raise cls.skipException("Floating ips are not available") |
Emily Hugenbruch | 5e2d2a2 | 2015-02-25 21:35:45 +0000 | [diff] [blame] | 95 | |
| 96 | @classmethod |
| 97 | def setup_credentials(cls): |
Masayuki Igawa | 60ea6c5 | 2014-10-15 17:32:14 +0900 | [diff] [blame] | 98 | # Create no network resources for these tests. |
| 99 | cls.set_network_resources() |
Emily Hugenbruch | 5e2d2a2 | 2015-02-25 21:35:45 +0000 | [diff] [blame] | 100 | super(TestNetworkBasicOps, cls).setup_credentials() |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 101 | |
Yair Fried | ed8392f | 2014-01-15 17:21:35 +0200 | [diff] [blame] | 102 | def setUp(self): |
| 103 | super(TestNetworkBasicOps, self).setUp() |
Yair Fried | 1fc32a1 | 2014-08-04 09:11:30 +0300 | [diff] [blame] | 104 | self.keypairs = {} |
| 105 | self.servers = [] |
David Shrewsbury | 9bac366 | 2014-08-07 15:07:01 -0400 | [diff] [blame] | 106 | |
Yair Fried | 413bf2d | 2014-11-19 17:07:11 +0200 | [diff] [blame] | 107 | def _setup_network_and_servers(self, **kwargs): |
Matt Riedemann | 1794073 | 2015-03-13 14:18:19 +0000 | [diff] [blame] | 108 | boot_with_port = kwargs.pop('boot_with_port', False) |
Yair Fried | 413bf2d | 2014-11-19 17:07:11 +0200 | [diff] [blame] | 109 | self.network, self.subnet, self.router = self.create_networks(**kwargs) |
David Shrewsbury | 9bac366 | 2014-08-07 15:07:01 -0400 | [diff] [blame] | 110 | self.check_networks() |
| 111 | |
Shuquan Huang | b5c8beb | 2015-08-05 14:14:01 +0000 | [diff] [blame] | 112 | self.ports = [] |
zhufl | 7a8f29d | 2017-02-17 10:16:45 +0800 | [diff] [blame] | 113 | port_id = None |
Jordan Pittier | f72a1dc | 2015-08-04 12:50:33 +0000 | [diff] [blame] | 114 | if boot_with_port: |
Matt Riedemann | 1794073 | 2015-03-13 14:18:19 +0000 | [diff] [blame] | 115 | # create a port on the network and boot with that |
zhufl | 1e446b5 | 2017-10-16 16:54:57 +0800 | [diff] [blame] | 116 | port_id = self.create_port(self.network['id'])['id'] |
zhufl | 7a8f29d | 2017-02-17 10:16:45 +0800 | [diff] [blame] | 117 | self.ports.append({'port': port_id}) |
Matt Riedemann | 1794073 | 2015-03-13 14:18:19 +0000 | [diff] [blame] | 118 | |
zhufl | 7a8f29d | 2017-02-17 10:16:45 +0800 | [diff] [blame] | 119 | server = self._create_server(self.network, port_id) |
zhufl | 420a019 | 2017-09-28 11:04:50 +0800 | [diff] [blame] | 120 | ssh_login = CONF.validation.image_ssh_user |
| 121 | for server in self.servers: |
| 122 | # call the common method in the parent class |
| 123 | self.check_tenant_network_connectivity( |
| 124 | server, ssh_login, self._get_server_key(server), |
| 125 | servers_for_debug=self.servers) |
Yair Fried | 2d2f3fe | 2014-02-24 16:19:20 +0200 | [diff] [blame] | 126 | |
Yair Fried | ae0e73d | 2014-11-24 11:56:26 +0200 | [diff] [blame] | 127 | floating_ip = self.create_floating_ip(server) |
| 128 | self.floating_ip_tuple = Floating_IP_tuple(floating_ip, server) |
Maru Newby | 81f07a0 | 2012-09-05 20:21:19 -0700 | [diff] [blame] | 129 | |
Yair Fried | ed8392f | 2014-01-15 17:21:35 +0200 | [diff] [blame] | 130 | def check_networks(self): |
Ken'ichi Ohmichi | 2e2ee19 | 2015-11-19 09:48:27 +0000 | [diff] [blame] | 131 | """Checks that we see the newly created network/subnet/router |
| 132 | |
| 133 | via checking the result of list_[networks,routers,subnets] |
Yair Fried | ed8392f | 2014-01-15 17:21:35 +0200 | [diff] [blame] | 134 | """ |
| 135 | |
jeremy.zhang | 5870ff1 | 2017-05-25 11:24:23 +0800 | [diff] [blame] | 136 | seen_nets = self.os_admin.networks_client.list_networks() |
Jordan Pittier | 64e6b44 | 2017-02-20 19:29:02 +0100 | [diff] [blame] | 137 | seen_names = [n['name'] for n in seen_nets['networks']] |
| 138 | seen_ids = [n['id'] for n in seen_nets['networks']] |
Steve Heyman | 33735f2 | 2016-05-24 09:28:08 -0500 | [diff] [blame] | 139 | self.assertIn(self.network['name'], seen_names) |
| 140 | self.assertIn(self.network['id'], seen_ids) |
Yair Fried | ed8392f | 2014-01-15 17:21:35 +0200 | [diff] [blame] | 141 | |
David Shrewsbury | 9bac366 | 2014-08-07 15:07:01 -0400 | [diff] [blame] | 142 | if self.subnet: |
jeremy.zhang | 5870ff1 | 2017-05-25 11:24:23 +0800 | [diff] [blame] | 143 | seen_subnets = self.os_admin.subnets_client.list_subnets() |
Jordan Pittier | 64e6b44 | 2017-02-20 19:29:02 +0100 | [diff] [blame] | 144 | seen_net_ids = [n['network_id'] for n in seen_subnets['subnets']] |
| 145 | seen_subnet_ids = [n['id'] for n in seen_subnets['subnets']] |
Steve Heyman | 33735f2 | 2016-05-24 09:28:08 -0500 | [diff] [blame] | 146 | self.assertIn(self.network['id'], seen_net_ids) |
| 147 | self.assertIn(self.subnet['id'], seen_subnet_ids) |
Yair Fried | ed8392f | 2014-01-15 17:21:35 +0200 | [diff] [blame] | 148 | |
David Shrewsbury | 9bac366 | 2014-08-07 15:07:01 -0400 | [diff] [blame] | 149 | if self.router: |
jeremy.zhang | 5870ff1 | 2017-05-25 11:24:23 +0800 | [diff] [blame] | 150 | seen_routers = self.os_admin.routers_client.list_routers() |
Jordan Pittier | 64e6b44 | 2017-02-20 19:29:02 +0100 | [diff] [blame] | 151 | seen_router_ids = [n['id'] for n in seen_routers['routers']] |
| 152 | seen_router_names = [n['name'] for n in seen_routers['routers']] |
Steve Heyman | 33735f2 | 2016-05-24 09:28:08 -0500 | [diff] [blame] | 153 | self.assertIn(self.router['name'], |
David Shrewsbury | 9bac366 | 2014-08-07 15:07:01 -0400 | [diff] [blame] | 154 | seen_router_names) |
Steve Heyman | 33735f2 | 2016-05-24 09:28:08 -0500 | [diff] [blame] | 155 | self.assertIn(self.router['id'], |
David Shrewsbury | 9bac366 | 2014-08-07 15:07:01 -0400 | [diff] [blame] | 156 | seen_router_ids) |
Gavin Brebner | 851c350 | 2013-01-18 13:14:10 +0000 | [diff] [blame] | 157 | |
zhufl | 24208c2 | 2016-10-25 15:23:48 +0800 | [diff] [blame] | 158 | def _create_server(self, network, port_id=None): |
Yair Fried | 1fc32a1 | 2014-08-04 09:11:30 +0300 | [diff] [blame] | 159 | keypair = self.create_keypair() |
| 160 | self.keypairs[keypair['name']] = keypair |
Jordan Pittier | 37b94a0 | 2017-02-21 18:11:55 +0100 | [diff] [blame] | 161 | security_groups = [ |
| 162 | {'name': self._create_security_group()['name']} |
| 163 | ] |
Steve Heyman | 33735f2 | 2016-05-24 09:28:08 -0500 | [diff] [blame] | 164 | network = {'uuid': network['id']} |
Matt Riedemann | 1794073 | 2015-03-13 14:18:19 +0000 | [diff] [blame] | 165 | if port_id is not None: |
lanoux | 5fc1452 | 2015-09-21 08:17:35 +0000 | [diff] [blame] | 166 | network['port'] = port_id |
| 167 | |
| 168 | server = self.create_server( |
lanoux | 5fc1452 | 2015-09-21 08:17:35 +0000 | [diff] [blame] | 169 | networks=[network], |
| 170 | key_name=keypair['name'], |
zhufl | 13c9c89 | 2017-02-10 12:04:07 +0800 | [diff] [blame] | 171 | security_groups=security_groups) |
Yair Fried | 1fc32a1 | 2014-08-04 09:11:30 +0300 | [diff] [blame] | 172 | self.servers.append(server) |
| 173 | return server |
| 174 | |
| 175 | def _get_server_key(self, server): |
| 176 | return self.keypairs[server['key_name']]['private_key'] |
Salvatore Orlando | 5ed3b6e | 2013-09-17 01:27:26 -0700 | [diff] [blame] | 177 | |
zhufl | 0ec74c4 | 2017-11-15 14:02:28 +0800 | [diff] [blame] | 178 | def _check_public_network_connectivity( |
Alok Maurya | 6384bbb | 2014-07-13 06:44:29 -0700 | [diff] [blame] | 179 | self, should_connect=True, msg=None, |
Ihar Hrachyshka | f9227c0 | 2016-09-15 11:16:47 +0000 | [diff] [blame] | 180 | should_check_floating_ip_status=True, mtu=None): |
Ken'ichi Ohmichi | 2e2ee19 | 2015-11-19 09:48:27 +0000 | [diff] [blame] | 181 | """Verifies connectivty to a VM via public network and floating IP |
| 182 | |
Yair Fried | 45f9295 | 2014-06-26 05:19:19 +0300 | [diff] [blame] | 183 | and verifies floating IP has resource status is correct. |
| 184 | |
Yair Fried | 45f9295 | 2014-06-26 05:19:19 +0300 | [diff] [blame] | 185 | :param should_connect: bool. determines if connectivity check is |
| 186 | negative or positive. |
| 187 | :param msg: Failure message to add to Error message. Should describe |
| 188 | the place in the test scenario where the method was called, |
| 189 | to indicate the context of the failure |
Alok Maurya | 6384bbb | 2014-07-13 06:44:29 -0700 | [diff] [blame] | 190 | :param should_check_floating_ip_status: bool. should status of |
| 191 | floating_ip be checked or not |
Ihar Hrachyshka | f9227c0 | 2016-09-15 11:16:47 +0000 | [diff] [blame] | 192 | :param mtu: int. MTU network to use for connectivity validation |
Yair Fried | 45f9295 | 2014-06-26 05:19:19 +0300 | [diff] [blame] | 193 | """ |
lanoux | 283273b | 2015-12-04 03:01:54 -0800 | [diff] [blame] | 194 | ssh_login = CONF.validation.image_ssh_user |
Yair Fried | 2d2f3fe | 2014-02-24 16:19:20 +0200 | [diff] [blame] | 195 | floating_ip, server = self.floating_ip_tuple |
Steve Heyman | 33735f2 | 2016-05-24 09:28:08 -0500 | [diff] [blame] | 196 | ip_address = floating_ip['floating_ip_address'] |
Yair Fried | 2d2f3fe | 2014-02-24 16:19:20 +0200 | [diff] [blame] | 197 | private_key = None |
Yair Fried | 45f9295 | 2014-06-26 05:19:19 +0300 | [diff] [blame] | 198 | floatingip_status = 'DOWN' |
Yair Fried | 2d2f3fe | 2014-02-24 16:19:20 +0200 | [diff] [blame] | 199 | if should_connect: |
Yair Fried | 1fc32a1 | 2014-08-04 09:11:30 +0300 | [diff] [blame] | 200 | private_key = self._get_server_key(server) |
Yair Fried | 45f9295 | 2014-06-26 05:19:19 +0300 | [diff] [blame] | 201 | floatingip_status = 'ACTIVE' |
zhufl | 0ec74c4 | 2017-11-15 14:02:28 +0800 | [diff] [blame] | 202 | |
Swaminathan Vasudevan | dc8bcdb | 2015-02-28 12:47:21 -0800 | [diff] [blame] | 203 | # Check FloatingIP Status before initiating a connection |
| 204 | if should_check_floating_ip_status: |
| 205 | self.check_floating_ip_status(floating_ip, floatingip_status) |
zhufl | 0ec74c4 | 2017-11-15 14:02:28 +0800 | [diff] [blame] | 206 | |
| 207 | message = 'Public network connectivity check failed' |
| 208 | if msg: |
| 209 | message += '. Reason: %s' % msg |
| 210 | |
| 211 | self.check_vm_connectivity( |
| 212 | ip_address, ssh_login, private_key, should_connect, |
| 213 | message, server, mtu=mtu) |
Matthew Treinish | 2b59f84 | 2013-09-09 20:32:51 +0000 | [diff] [blame] | 214 | |
Yair Fried | 9a551c4 | 2013-12-15 14:59:34 +0200 | [diff] [blame] | 215 | def _disassociate_floating_ips(self): |
Ferenc Horváth | bce1fcf | 2017-06-07 11:19:51 +0200 | [diff] [blame] | 216 | floating_ip, _ = self.floating_ip_tuple |
zhufl | afe96b8 | 2017-09-22 14:25:29 +0800 | [diff] [blame] | 217 | floating_ip = self.floating_ips_client.update_floatingip( |
| 218 | floating_ip['id'], port_id=None)['floatingip'] |
| 219 | self.assertIsNone(floating_ip['port_id']) |
| 220 | self.floating_ip_tuple = Floating_IP_tuple(floating_ip, None) |
Yair Fried | 9a551c4 | 2013-12-15 14:59:34 +0200 | [diff] [blame] | 221 | |
Yair Fried | 05db252 | 2013-11-18 11:02:10 +0200 | [diff] [blame] | 222 | def _reassociate_floating_ips(self): |
Yair Fried | 2d2f3fe | 2014-02-24 16:19:20 +0200 | [diff] [blame] | 223 | floating_ip, server = self.floating_ip_tuple |
Yair Fried | 2d2f3fe | 2014-02-24 16:19:20 +0200 | [diff] [blame] | 224 | # create a new server for the floating ip |
zhufl | 24208c2 | 2016-10-25 15:23:48 +0800 | [diff] [blame] | 225 | server = self._create_server(self.network) |
zhufl | afe96b8 | 2017-09-22 14:25:29 +0800 | [diff] [blame] | 226 | port_id, _ = self._get_server_port_id_and_ip4(server) |
| 227 | floating_ip = self.floating_ips_client.update_floatingip( |
| 228 | floating_ip['id'], port_id=port_id)['floatingip'] |
| 229 | self.assertEqual(port_id, floating_ip['port_id']) |
| 230 | self.floating_ip_tuple = Floating_IP_tuple(floating_ip, server) |
Yair Fried | 05db252 | 2013-11-18 11:02:10 +0200 | [diff] [blame] | 231 | |
Fei Long Wang | 05a1c4a | 2015-02-02 16:58:24 +1300 | [diff] [blame] | 232 | def _create_new_network(self, create_gateway=False): |
Marc Koderer | 410c782 | 2016-11-08 11:47:00 +0100 | [diff] [blame] | 233 | self.new_net = self._create_network() |
Fei Long Wang | 05a1c4a | 2015-02-02 16:58:24 +1300 | [diff] [blame] | 234 | if create_gateway: |
zhufl | 5b0a52f | 2017-10-24 15:48:20 +0800 | [diff] [blame] | 235 | self.new_subnet = self.create_subnet( |
Fei Long Wang | 05a1c4a | 2015-02-02 16:58:24 +1300 | [diff] [blame] | 236 | network=self.new_net) |
| 237 | else: |
zhufl | 5b0a52f | 2017-10-24 15:48:20 +0800 | [diff] [blame] | 238 | self.new_subnet = self.create_subnet( |
Fei Long Wang | 05a1c4a | 2015-02-02 16:58:24 +1300 | [diff] [blame] | 239 | network=self.new_net, |
| 240 | gateway_ip=None) |
Yair Fried | 3097dc1 | 2014-01-26 08:46:43 +0200 | [diff] [blame] | 241 | |
| 242 | def _hotplug_server(self): |
| 243 | old_floating_ip, server = self.floating_ip_tuple |
Steve Heyman | 33735f2 | 2016-05-24 09:28:08 -0500 | [diff] [blame] | 244 | ip_address = old_floating_ip['floating_ip_address'] |
Yair Fried | 1fc32a1 | 2014-08-04 09:11:30 +0300 | [diff] [blame] | 245 | private_key = self._get_server_key(server) |
Jordan Pittier | bbb1712 | 2016-01-26 17:10:55 +0100 | [diff] [blame] | 246 | ssh_client = self.get_remote_client( |
zhufl | f52c759 | 2017-05-25 13:55:24 +0800 | [diff] [blame] | 247 | ip_address, private_key=private_key, server=server) |
Yair Fried | 3097dc1 | 2014-01-26 08:46:43 +0200 | [diff] [blame] | 248 | old_nic_list = self._get_server_nics(ssh_client) |
| 249 | # get a port from a list of one item |
jeremy.zhang | 5870ff1 | 2017-05-25 11:24:23 +0800 | [diff] [blame] | 250 | port_list = self.os_admin.ports_client.list_ports( |
Jordan Pittier | 64e6b44 | 2017-02-20 19:29:02 +0100 | [diff] [blame] | 251 | device_id=server['id'])['ports'] |
Yair Fried | 3097dc1 | 2014-01-26 08:46:43 +0200 | [diff] [blame] | 252 | self.assertEqual(1, len(port_list)) |
| 253 | old_port = port_list[0] |
David Kranz | b2b0c18 | 2015-02-18 13:28:19 -0500 | [diff] [blame] | 254 | interface = self.interface_client.create_interface( |
Ken'ichi Ohmichi | 9509b96 | 2015-07-07 05:30:15 +0000 | [diff] [blame] | 255 | server_id=server['id'], |
Steve Heyman | 33735f2 | 2016-05-24 09:28:08 -0500 | [diff] [blame] | 256 | net_id=self.new_net['id'])['interfaceAttachment'] |
Ken'ichi Ohmichi | 43e7fcf | 2016-04-04 11:59:13 -0700 | [diff] [blame] | 257 | self.addCleanup(self.ports_client.wait_for_resource_deletion, |
| 258 | interface['port_id']) |
Jordan Pittier | 9e227c5 | 2016-02-09 14:35:18 +0100 | [diff] [blame] | 259 | self.addCleanup(test_utils.call_and_ignore_notfound_exc, |
Yair Fried | 1fc32a1 | 2014-08-04 09:11:30 +0300 | [diff] [blame] | 260 | self.interface_client.delete_interface, |
| 261 | server['id'], interface['port_id']) |
Yair Fried | 3097dc1 | 2014-01-26 08:46:43 +0200 | [diff] [blame] | 262 | |
| 263 | def check_ports(): |
Jordan Pittier | 64e6b44 | 2017-02-20 19:29:02 +0100 | [diff] [blame] | 264 | self.new_port_list = [ |
| 265 | port for port in |
jeremy.zhang | 5870ff1 | 2017-05-25 11:24:23 +0800 | [diff] [blame] | 266 | self.os_admin.ports_client.list_ports( |
Jordan Pittier | 64e6b44 | 2017-02-20 19:29:02 +0100 | [diff] [blame] | 267 | device_id=server['id'])['ports'] |
| 268 | if port['id'] != old_port['id'] |
| 269 | ] |
Attila Fazekas | a8bb394 | 2014-08-19 09:06:30 +0200 | [diff] [blame] | 270 | return len(self.new_port_list) == 1 |
Yair Fried | 3097dc1 | 2014-01-26 08:46:43 +0200 | [diff] [blame] | 271 | |
Jordan Pittier | 35a6375 | 2016-08-30 13:09:12 +0200 | [diff] [blame] | 272 | if not test_utils.call_until_true( |
| 273 | check_ports, CONF.network.build_timeout, |
| 274 | CONF.network.build_interval): |
Matt Riedemann | 892094e | 2015-02-05 07:24:02 -0800 | [diff] [blame] | 275 | raise exceptions.TimeoutException( |
| 276 | "No new port attached to the server in time (%s sec)! " |
| 277 | "Old port: %s. Number of new ports: %d" % ( |
| 278 | CONF.network.build_timeout, old_port, |
| 279 | len(self.new_port_list))) |
Steve Heyman | 33735f2 | 2016-05-24 09:28:08 -0500 | [diff] [blame] | 280 | new_port = self.new_port_list[0] |
Attila Fazekas | a8bb394 | 2014-08-19 09:06:30 +0200 | [diff] [blame] | 281 | |
| 282 | def check_new_nic(): |
| 283 | new_nic_list = self._get_server_nics(ssh_client) |
| 284 | self.diff_list = [n for n in new_nic_list if n not in old_nic_list] |
| 285 | return len(self.diff_list) == 1 |
| 286 | |
Jordan Pittier | 35a6375 | 2016-08-30 13:09:12 +0200 | [diff] [blame] | 287 | if not test_utils.call_until_true( |
| 288 | check_new_nic, CONF.network.build_timeout, |
| 289 | CONF.network.build_interval): |
Attila Fazekas | a8bb394 | 2014-08-19 09:06:30 +0200 | [diff] [blame] | 290 | raise exceptions.TimeoutException("Interface not visible on the " |
| 291 | "guest after %s sec" |
| 292 | % CONF.network.build_timeout) |
| 293 | |
Ferenc Horváth | bce1fcf | 2017-06-07 11:19:51 +0200 | [diff] [blame] | 294 | _, new_nic = self.diff_list[0] |
Martin Kopec | 925188d | 2019-05-07 13:26:34 +0000 | [diff] [blame] | 295 | ip_output = ssh_client.exec_command('ip a') |
| 296 | ip_address = new_port['fixed_ips'][0]['ip_address'] |
| 297 | ip_mask = CONF.network.project_network_mask_bits |
| 298 | # check if the address is not already in use, if not, set it |
| 299 | if ' ' + ip_address + '/' + str(ip_mask) not in ip_output: |
Rodolfo Alonso Hernandez | 15d6309 | 2020-03-31 15:34:30 +0000 | [diff] [blame] | 300 | try: |
| 301 | ssh_client.exec_command("sudo ip addr add %s/%s dev %s" % ( |
| 302 | ip_address, ip_mask, new_nic)) |
| 303 | ssh_client.exec_command("sudo ip link set %s up" % new_nic) |
| 304 | except exceptions.SSHExecCommandFailed as exc: |
| 305 | if 'RTNETLINK answers: File exists' in str(exc): |
| 306 | LOG.debug( |
| 307 | 'IP address %(ip_address)s is already set in device ' |
| 308 | '%(device)s\nPrevious "ip a" output: %(ip_output)s', |
| 309 | {'ip_address': ip_address, 'device': new_nic, |
| 310 | 'ip_output': ip_output}) |
| 311 | else: |
| 312 | raise exc |
Yair Fried | 3097dc1 | 2014-01-26 08:46:43 +0200 | [diff] [blame] | 313 | |
| 314 | def _get_server_nics(self, ssh_client): |
James Page | a936627 | 2017-06-09 12:05:09 +0100 | [diff] [blame] | 315 | reg = re.compile(r'(?P<num>\d+): (?P<nic_name>\w+)[@]?.*:') |
Ken'ichi Ohmichi | 84aeba6 | 2017-03-01 18:31:20 -0800 | [diff] [blame] | 316 | ipatxt = ssh_client.exec_command("ip address") |
Yair Fried | 3097dc1 | 2014-01-26 08:46:43 +0200 | [diff] [blame] | 317 | return reg.findall(ipatxt) |
| 318 | |
Fei Long Wang | 05a1c4a | 2015-02-02 16:58:24 +1300 | [diff] [blame] | 319 | def _check_network_internal_connectivity(self, network, |
| 320 | should_connect=True): |
Ken'ichi Ohmichi | 2e2ee19 | 2015-11-19 09:48:27 +0000 | [diff] [blame] | 321 | """via ssh check VM internal connectivity: |
| 322 | |
Yair Fried | 0655229 | 2013-11-11 12:10:09 +0200 | [diff] [blame] | 323 | - ping internal gateway and DHCP port, implying in-tenant connectivity |
| 324 | pinging both, because L3 and DHCP agents might be on different nodes |
Tong Liu | 589f315 | 2016-03-24 05:56:40 +0000 | [diff] [blame] | 325 | - ping internal compute port, implying connectivity to other VMs on |
| 326 | this network |
Yair Fried | 3097dc1 | 2014-01-26 08:46:43 +0200 | [diff] [blame] | 327 | """ |
| 328 | floating_ip, server = self.floating_ip_tuple |
| 329 | # get internal ports' ips: |
Tong Liu | 589f315 | 2016-03-24 05:56:40 +0000 | [diff] [blame] | 330 | # get all network and compute ports in the new network |
Jordan Pittier | 64e6b44 | 2017-02-20 19:29:02 +0100 | [diff] [blame] | 331 | internal_ips = ( |
| 332 | p['fixed_ips'][0]['ip_address'] for p in |
jeremy.zhang | 5870ff1 | 2017-05-25 11:24:23 +0800 | [diff] [blame] | 333 | self.os_admin.ports_client.list_ports( |
Rodolfo Alonso Hernandez | c1449d4 | 2020-02-15 13:24:28 +0000 | [diff] [blame] | 334 | project_id=server['tenant_id'], |
Jordan Pittier | 64e6b44 | 2017-02-20 19:29:02 +0100 | [diff] [blame] | 335 | network_id=network['id'])['ports'] |
Tong Liu | 589f315 | 2016-03-24 05:56:40 +0000 | [diff] [blame] | 336 | if p['device_owner'].startswith('network') or |
| 337 | p['device_owner'].startswith('compute') |
Jordan Pittier | 64e6b44 | 2017-02-20 19:29:02 +0100 | [diff] [blame] | 338 | ) |
Yair Fried | 3097dc1 | 2014-01-26 08:46:43 +0200 | [diff] [blame] | 339 | |
Fei Long Wang | 05a1c4a | 2015-02-02 16:58:24 +1300 | [diff] [blame] | 340 | self._check_server_connectivity(floating_ip, |
| 341 | internal_ips, |
| 342 | should_connect) |
Yair Fried | 0655229 | 2013-11-11 12:10:09 +0200 | [diff] [blame] | 343 | |
| 344 | def _check_network_external_connectivity(self): |
Ken'ichi Ohmichi | 2e2ee19 | 2015-11-19 09:48:27 +0000 | [diff] [blame] | 345 | """ping default gateway to imply external connectivity""" |
Yair Fried | 0655229 | 2013-11-11 12:10:09 +0200 | [diff] [blame] | 346 | if not CONF.network.public_network_id: |
| 347 | msg = 'public network not defined.' |
| 348 | LOG.info(msg) |
| 349 | return |
| 350 | |
Andrew Boik | 4a3daf1 | 2015-03-27 01:59:31 -0400 | [diff] [blame] | 351 | # We ping the external IP from the instance using its floating IP |
| 352 | # which is always IPv4, so we must only test connectivity to |
| 353 | # external IPv4 IPs if the external network is dualstack. |
Jordan Pittier | 64e6b44 | 2017-02-20 19:29:02 +0100 | [diff] [blame] | 354 | v4_subnets = [ |
jeremy.zhang | 5870ff1 | 2017-05-25 11:24:23 +0800 | [diff] [blame] | 355 | s for s in self.os_admin.subnets_client.list_subnets( |
Jordan Pittier | 64e6b44 | 2017-02-20 19:29:02 +0100 | [diff] [blame] | 356 | network_id=CONF.network.public_network_id)['subnets'] |
| 357 | if s['ip_version'] == 4 |
| 358 | ] |
Yair Fried | 0655229 | 2013-11-11 12:10:09 +0200 | [diff] [blame] | 359 | |
Lukas Piwowarski | 2385e04 | 2020-01-31 12:28:20 +0000 | [diff] [blame] | 360 | if len(v4_subnets) > 1: |
| 361 | self.assertTrue( |
| 362 | CONF.network.subnet_id, |
| 363 | "Found %d subnets. Specify subnet using configuration " |
| 364 | "option [network].subnet_id." |
| 365 | % len(v4_subnets)) |
| 366 | subnet = self.os_admin.subnets_client.show_subnet( |
| 367 | CONF.network.subnet_id)['subnet'] |
| 368 | external_ips = [subnet['gateway_ip']] |
| 369 | else: |
| 370 | external_ips = [v4_subnets[0]['gateway_ip']] |
| 371 | |
Yair Fried | 0655229 | 2013-11-11 12:10:09 +0200 | [diff] [blame] | 372 | self._check_server_connectivity(self.floating_ip_tuple.floating_ip, |
| 373 | external_ips) |
| 374 | |
Fei Long Wang | 05a1c4a | 2015-02-02 16:58:24 +1300 | [diff] [blame] | 375 | def _check_server_connectivity(self, floating_ip, address_list, |
| 376 | should_connect=True): |
Steve Heyman | 33735f2 | 2016-05-24 09:28:08 -0500 | [diff] [blame] | 377 | ip_address = floating_ip['floating_ip_address'] |
Yair Fried | 1fc32a1 | 2014-08-04 09:11:30 +0300 | [diff] [blame] | 378 | private_key = self._get_server_key(self.floating_ip_tuple.server) |
Jordan Pittier | bbb1712 | 2016-01-26 17:10:55 +0100 | [diff] [blame] | 379 | ssh_source = self.get_remote_client( |
zhufl | f52c759 | 2017-05-25 13:55:24 +0800 | [diff] [blame] | 380 | ip_address, private_key=private_key, |
| 381 | server=self.floating_ip_tuple.server) |
Yair Fried | 3097dc1 | 2014-01-26 08:46:43 +0200 | [diff] [blame] | 382 | |
Yair Fried | 0655229 | 2013-11-11 12:10:09 +0200 | [diff] [blame] | 383 | for remote_ip in address_list: |
YAMAMOTO Takashi | 4c3ebb0 | 2017-01-25 16:04:30 +0900 | [diff] [blame] | 384 | self.check_remote_connectivity(ssh_source, remote_ip, |
| 385 | should_connect) |
Yair Fried | 3097dc1 | 2014-01-26 08:46:43 +0200 | [diff] [blame] | 386 | |
zhufl | afe96b8 | 2017-09-22 14:25:29 +0800 | [diff] [blame] | 387 | def _update_router_admin_state(self, router, admin_state_up): |
| 388 | kwargs = dict(admin_state_up=admin_state_up) |
| 389 | router = self.routers_client.update_router( |
| 390 | router['id'], **kwargs)['router'] |
| 391 | self.assertEqual(admin_state_up, router['admin_state_up']) |
| 392 | |
Jordan Pittier | 3b46d27 | 2017-04-12 16:17:28 +0200 | [diff] [blame] | 393 | @decorators.attr(type='smoke') |
Ken'ichi Ohmichi | c85a951 | 2017-01-27 18:34:24 -0800 | [diff] [blame] | 394 | @decorators.idempotent_id('f323b3ba-82f8-4db7-8ea6-6a895869ec49') |
Andrea Frittoli | cd36841 | 2017-08-14 21:37:56 +0100 | [diff] [blame] | 395 | @utils.services('compute', 'network') |
Matthew Treinish | 2b59f84 | 2013-09-09 20:32:51 +0000 | [diff] [blame] | 396 | def test_network_basic_ops(self): |
Ken'ichi Ohmichi | 2e2ee19 | 2015-11-19 09:48:27 +0000 | [diff] [blame] | 397 | """Basic network operation test |
| 398 | |
Sergey Vilgelm | eac094a | 2018-11-21 18:27:51 -0600 | [diff] [blame] | 399 | For a freshly-booted VM with an IP address ("port") on a given network: |
Yair Fried | 3097dc1 | 2014-01-26 08:46:43 +0200 | [diff] [blame] | 400 | |
| 401 | - the Tempest host can ping the IP address. This implies, but |
Sergey Vilgelm | eac094a | 2018-11-21 18:27:51 -0600 | [diff] [blame] | 402 | does not guarantee (see the ssh check that follows), that the |
| 403 | VM has been assigned the correct IP address and has |
| 404 | connectivity to the Tempest host. |
Yair Fried | 3097dc1 | 2014-01-26 08:46:43 +0200 | [diff] [blame] | 405 | |
| 406 | - the Tempest host can perform key-based authentication to an |
Sergey Vilgelm | eac094a | 2018-11-21 18:27:51 -0600 | [diff] [blame] | 407 | ssh server hosted at the IP address. This check guarantees |
| 408 | that the IP address is associated with the target VM. |
Yair Fried | 3097dc1 | 2014-01-26 08:46:43 +0200 | [diff] [blame] | 409 | |
Yair Fried | 3097dc1 | 2014-01-26 08:46:43 +0200 | [diff] [blame] | 410 | - the Tempest host can ssh into the VM via the IP address and |
Sergey Vilgelm | eac094a | 2018-11-21 18:27:51 -0600 | [diff] [blame] | 411 | successfully execute the following: |
Yair Fried | 3097dc1 | 2014-01-26 08:46:43 +0200 | [diff] [blame] | 412 | |
Sergey Vilgelm | eac094a | 2018-11-21 18:27:51 -0600 | [diff] [blame] | 413 | - ping an external IP address, implying external connectivity. |
Yair Fried | 3097dc1 | 2014-01-26 08:46:43 +0200 | [diff] [blame] | 414 | |
Sergey Vilgelm | eac094a | 2018-11-21 18:27:51 -0600 | [diff] [blame] | 415 | - ping an external hostname, implying that dns is correctly |
| 416 | configured. |
Yair Fried | 3097dc1 | 2014-01-26 08:46:43 +0200 | [diff] [blame] | 417 | |
Sergey Vilgelm | eac094a | 2018-11-21 18:27:51 -0600 | [diff] [blame] | 418 | - ping an internal IP address, implying connectivity to another |
| 419 | VM on the same network. |
Yair Fried | 3097dc1 | 2014-01-26 08:46:43 +0200 | [diff] [blame] | 420 | |
Yair Fried | 0655229 | 2013-11-11 12:10:09 +0200 | [diff] [blame] | 421 | - detach the floating-ip from the VM and verify that it becomes |
Sergey Vilgelm | eac094a | 2018-11-21 18:27:51 -0600 | [diff] [blame] | 422 | unreachable |
Yair Fried | 0655229 | 2013-11-11 12:10:09 +0200 | [diff] [blame] | 423 | |
| 424 | - associate detached floating ip to a new VM and verify connectivity. |
Sergey Vilgelm | eac094a | 2018-11-21 18:27:51 -0600 | [diff] [blame] | 425 | VMs are created with unique keypair so connectivity also asserts |
| 426 | that floating IP is associated with the new VM instead of the old |
| 427 | one |
Yair Fried | 0655229 | 2013-11-11 12:10:09 +0200 | [diff] [blame] | 428 | |
Yair Fried | 45f9295 | 2014-06-26 05:19:19 +0300 | [diff] [blame] | 429 | Verifies that floating IP status is updated correctly after each change |
Yair Fried | 3097dc1 | 2014-01-26 08:46:43 +0200 | [diff] [blame] | 430 | """ |
David Shrewsbury | 9bac366 | 2014-08-07 15:07:01 -0400 | [diff] [blame] | 431 | self._setup_network_and_servers() |
zhufl | 0ec74c4 | 2017-11-15 14:02:28 +0800 | [diff] [blame] | 432 | self._check_public_network_connectivity(should_connect=True) |
Yair Fried | 0655229 | 2013-11-11 12:10:09 +0200 | [diff] [blame] | 433 | self._check_network_internal_connectivity(network=self.network) |
| 434 | self._check_network_external_connectivity() |
Yair Fried | 9a551c4 | 2013-12-15 14:59:34 +0200 | [diff] [blame] | 435 | self._disassociate_floating_ips() |
zhufl | 0ec74c4 | 2017-11-15 14:02:28 +0800 | [diff] [blame] | 436 | self._check_public_network_connectivity(should_connect=False, |
| 437 | msg="after disassociate " |
| 438 | "floating ip") |
Yair Fried | 05db252 | 2013-11-18 11:02:10 +0200 | [diff] [blame] | 439 | self._reassociate_floating_ips() |
zhufl | 0ec74c4 | 2017-11-15 14:02:28 +0800 | [diff] [blame] | 440 | self._check_public_network_connectivity(should_connect=True, |
| 441 | msg="after re-associate " |
| 442 | "floating ip") |
Yair Fried | 3097dc1 | 2014-01-26 08:46:43 +0200 | [diff] [blame] | 443 | |
Ken'ichi Ohmichi | c85a951 | 2017-01-27 18:34:24 -0800 | [diff] [blame] | 444 | @decorators.idempotent_id('b158ea55-472e-4086-8fa9-c64ac0c6c1d0') |
Andrea Frittoli | cd36841 | 2017-08-14 21:37:56 +0100 | [diff] [blame] | 445 | @testtools.skipUnless(utils.is_extension_enabled('net-mtu', 'network'), |
Ihar Hrachyshka | f9227c0 | 2016-09-15 11:16:47 +0000 | [diff] [blame] | 446 | 'No way to calculate MTU for networks') |
Jordan Pittier | 3b46d27 | 2017-04-12 16:17:28 +0200 | [diff] [blame] | 447 | @decorators.attr(type='slow') |
Andrea Frittoli | cd36841 | 2017-08-14 21:37:56 +0100 | [diff] [blame] | 448 | @utils.services('compute', 'network') |
Ihar Hrachyshka | f9227c0 | 2016-09-15 11:16:47 +0000 | [diff] [blame] | 449 | def test_mtu_sized_frames(self): |
| 450 | """Validate that network MTU sized frames fit through.""" |
| 451 | self._setup_network_and_servers() |
Ihar Hrachyshka | aca29ed | 2017-09-25 12:13:53 -0700 | [diff] [blame] | 452 | # first check that connectivity works in general for the instance |
zhufl | 0ec74c4 | 2017-11-15 14:02:28 +0800 | [diff] [blame] | 453 | self._check_public_network_connectivity(should_connect=True) |
Ihar Hrachyshka | aca29ed | 2017-09-25 12:13:53 -0700 | [diff] [blame] | 454 | # now that we checked general connectivity, test that full size frames |
| 455 | # can also pass between nodes |
zhufl | 0ec74c4 | 2017-11-15 14:02:28 +0800 | [diff] [blame] | 456 | self._check_public_network_connectivity( |
Ihar Hrachyshka | f9227c0 | 2016-09-15 11:16:47 +0000 | [diff] [blame] | 457 | should_connect=True, mtu=self.network['mtu']) |
| 458 | |
Ken'ichi Ohmichi | c85a951 | 2017-01-27 18:34:24 -0800 | [diff] [blame] | 459 | @decorators.idempotent_id('1546850e-fbaa-42f5-8b5f-03d8a6a95f15') |
Thiago Paiva | 66cded2 | 2016-08-15 14:55:58 -0300 | [diff] [blame] | 460 | @testtools.skipIf(CONF.network.shared_physical_network, |
| 461 | 'Connectivity can only be tested when in a ' |
| 462 | 'multitenant network environment') |
Jordan Pittier | 3b46d27 | 2017-04-12 16:17:28 +0200 | [diff] [blame] | 463 | @decorators.attr(type='slow') |
Andrea Frittoli | cd36841 | 2017-08-14 21:37:56 +0100 | [diff] [blame] | 464 | @utils.services('compute', 'network') |
Fei Long Wang | 05a1c4a | 2015-02-02 16:58:24 +1300 | [diff] [blame] | 465 | def test_connectivity_between_vms_on_different_networks(self): |
Ken'ichi Ohmichi | 2e2ee19 | 2015-11-19 09:48:27 +0000 | [diff] [blame] | 466 | """Test connectivity between VMs on different networks |
| 467 | |
Sergey Vilgelm | eac094a | 2018-11-21 18:27:51 -0600 | [diff] [blame] | 468 | For a freshly-booted VM with an IP address ("port") on a given network: |
Fei Long Wang | 05a1c4a | 2015-02-02 16:58:24 +1300 | [diff] [blame] | 469 | |
| 470 | - the Tempest host can ping the IP address. |
Fei Long Wang | 05a1c4a | 2015-02-02 16:58:24 +1300 | [diff] [blame] | 471 | - the Tempest host can ssh into the VM via the IP address and |
Sergey Vilgelm | eac094a | 2018-11-21 18:27:51 -0600 | [diff] [blame] | 472 | successfully execute the following: |
Fei Long Wang | 05a1c4a | 2015-02-02 16:58:24 +1300 | [diff] [blame] | 473 | |
Sergey Vilgelm | eac094a | 2018-11-21 18:27:51 -0600 | [diff] [blame] | 474 | - ping an external IP address, implying external connectivity. |
| 475 | - ping an external hostname, implying that dns is correctly |
| 476 | configured. |
| 477 | - ping an internal IP address, implying connectivity to another |
| 478 | VM on the same network. |
Fei Long Wang | 05a1c4a | 2015-02-02 16:58:24 +1300 | [diff] [blame] | 479 | |
| 480 | - Create another network on the same tenant with subnet, create |
Sergey Vilgelm | eac094a | 2018-11-21 18:27:51 -0600 | [diff] [blame] | 481 | an VM on the new network. |
Fei Long Wang | 05a1c4a | 2015-02-02 16:58:24 +1300 | [diff] [blame] | 482 | |
Sergey Vilgelm | eac094a | 2018-11-21 18:27:51 -0600 | [diff] [blame] | 483 | - Ping the new VM from previous VM failed since the new network |
| 484 | was not attached to router yet. |
| 485 | - Attach the new network to the router, Ping the new VM from |
| 486 | previous VM succeed. |
Fei Long Wang | 05a1c4a | 2015-02-02 16:58:24 +1300 | [diff] [blame] | 487 | |
| 488 | """ |
| 489 | self._setup_network_and_servers() |
zhufl | 0ec74c4 | 2017-11-15 14:02:28 +0800 | [diff] [blame] | 490 | self._check_public_network_connectivity(should_connect=True) |
Fei Long Wang | 05a1c4a | 2015-02-02 16:58:24 +1300 | [diff] [blame] | 491 | self._check_network_internal_connectivity(network=self.network) |
| 492 | self._check_network_external_connectivity() |
| 493 | self._create_new_network(create_gateway=True) |
Lucas Alvares Gomes | d595c36 | 2018-10-16 14:38:11 +0100 | [diff] [blame] | 494 | new_server = self._create_server(self.new_net) |
| 495 | new_server_ips = [addr['addr'] for addr in |
| 496 | new_server['addresses'][self.new_net['name']]] |
| 497 | |
| 498 | # Assert that pinging the new VM fails since the new network is not |
| 499 | # connected to a router |
| 500 | self._check_server_connectivity(self.floating_ip_tuple.floating_ip, |
| 501 | new_server_ips, should_connect=False) |
Steve Heyman | 33735f2 | 2016-05-24 09:28:08 -0500 | [diff] [blame] | 502 | router_id = self.router['id'] |
| 503 | self.routers_client.add_router_interface( |
| 504 | router_id, subnet_id=self.new_subnet['id']) |
| 505 | |
| 506 | self.addCleanup(test_utils.call_and_ignore_notfound_exc, |
| 507 | self.routers_client.remove_router_interface, |
| 508 | router_id, subnet_id=self.new_subnet['id']) |
Lucas Alvares Gomes | d595c36 | 2018-10-16 14:38:11 +0100 | [diff] [blame] | 509 | |
| 510 | self._check_server_connectivity(self.floating_ip_tuple.floating_ip, |
| 511 | new_server_ips, should_connect=True) |
Fei Long Wang | 05a1c4a | 2015-02-02 16:58:24 +1300 | [diff] [blame] | 512 | |
Ken'ichi Ohmichi | c85a951 | 2017-01-27 18:34:24 -0800 | [diff] [blame] | 513 | @decorators.idempotent_id('c5adff73-e961-41f1-b4a9-343614f18cfa') |
Adam Gandelman | 7186f7a | 2014-07-23 09:28:56 -0400 | [diff] [blame] | 514 | @testtools.skipUnless(CONF.compute_feature_enabled.interface_attach, |
| 515 | 'NIC hotplug not available') |
Itzik Brown | 2ca01cd | 2014-12-08 12:58:20 +0200 | [diff] [blame] | 516 | @testtools.skipIf(CONF.network.port_vnic_type in ['direct', 'macvtap'], |
| 517 | 'NIC hotplug not supported for ' |
| 518 | 'vnic_type direct or macvtap') |
Andrea Frittoli | cd36841 | 2017-08-14 21:37:56 +0100 | [diff] [blame] | 519 | @utils.services('compute', 'network') |
Yair Fried | 3097dc1 | 2014-01-26 08:46:43 +0200 | [diff] [blame] | 520 | def test_hotplug_nic(self): |
Ken'ichi Ohmichi | 2e2ee19 | 2015-11-19 09:48:27 +0000 | [diff] [blame] | 521 | """Test hotplug network interface |
| 522 | |
Arie | f3f6e81 | 2016-09-28 16:48:20 +0300 | [diff] [blame] | 523 | 1. Create a network and a VM. |
| 524 | 2. Check connectivity to the VM via a public network. |
| 525 | 3. Create a new network, with no gateway. |
| 526 | 4. Bring up a new interface |
| 527 | 5. check the VM reach the new network |
Yair Fried | 3097dc1 | 2014-01-26 08:46:43 +0200 | [diff] [blame] | 528 | |
| 529 | """ |
David Shrewsbury | 9bac366 | 2014-08-07 15:07:01 -0400 | [diff] [blame] | 530 | self._setup_network_and_servers() |
zhufl | 0ec74c4 | 2017-11-15 14:02:28 +0800 | [diff] [blame] | 531 | self._check_public_network_connectivity(should_connect=True) |
Yair Fried | 3097dc1 | 2014-01-26 08:46:43 +0200 | [diff] [blame] | 532 | self._create_new_network() |
| 533 | self._hotplug_server() |
Yair Fried | 0655229 | 2013-11-11 12:10:09 +0200 | [diff] [blame] | 534 | self._check_network_internal_connectivity(network=self.new_net) |
Alok Maurya | 6384bbb | 2014-07-13 06:44:29 -0700 | [diff] [blame] | 535 | |
Ken'ichi Ohmichi | c85a951 | 2017-01-27 18:34:24 -0800 | [diff] [blame] | 536 | @decorators.idempotent_id('04b9fe4e-85e8-4aea-b937-ea93885ac59f') |
Thiago Paiva | 66cded2 | 2016-08-15 14:55:58 -0300 | [diff] [blame] | 537 | @testtools.skipIf(CONF.network.shared_physical_network, |
| 538 | 'Router state can be altered only with multitenant ' |
| 539 | 'networks capabilities') |
Jordan Pittier | 3b46d27 | 2017-04-12 16:17:28 +0200 | [diff] [blame] | 540 | @decorators.attr(type='slow') |
Andrea Frittoli | cd36841 | 2017-08-14 21:37:56 +0100 | [diff] [blame] | 541 | @utils.services('compute', 'network') |
Alok Maurya | 6384bbb | 2014-07-13 06:44:29 -0700 | [diff] [blame] | 542 | def test_update_router_admin_state(self): |
Ken'ichi Ohmichi | 2e2ee19 | 2015-11-19 09:48:27 +0000 | [diff] [blame] | 543 | """Test to update admin state up of router |
| 544 | |
Alok Maurya | 6384bbb | 2014-07-13 06:44:29 -0700 | [diff] [blame] | 545 | 1. Check public connectivity before updating |
| 546 | admin_state_up attribute of router to False |
| 547 | 2. Check public connectivity after updating |
| 548 | admin_state_up attribute of router to False |
| 549 | 3. Check public connectivity after updating |
| 550 | admin_state_up attribute of router to True |
| 551 | """ |
| 552 | self._setup_network_and_servers() |
zhufl | 0ec74c4 | 2017-11-15 14:02:28 +0800 | [diff] [blame] | 553 | self._check_public_network_connectivity( |
Alok Maurya | 6384bbb | 2014-07-13 06:44:29 -0700 | [diff] [blame] | 554 | should_connect=True, msg="before updating " |
| 555 | "admin_state_up of router to False") |
| 556 | self._update_router_admin_state(self.router, False) |
| 557 | # TODO(alokmaurya): Remove should_check_floating_ip_status=False check |
| 558 | # once bug 1396310 is fixed |
| 559 | |
zhufl | 0ec74c4 | 2017-11-15 14:02:28 +0800 | [diff] [blame] | 560 | self._check_public_network_connectivity( |
Alok Maurya | 6384bbb | 2014-07-13 06:44:29 -0700 | [diff] [blame] | 561 | should_connect=False, msg="after updating " |
| 562 | "admin_state_up of router to False", |
| 563 | should_check_floating_ip_status=False) |
| 564 | self._update_router_admin_state(self.router, True) |
zhufl | 0ec74c4 | 2017-11-15 14:02:28 +0800 | [diff] [blame] | 565 | self._check_public_network_connectivity( |
Alok Maurya | 6384bbb | 2014-07-13 06:44:29 -0700 | [diff] [blame] | 566 | should_connect=True, msg="after updating " |
| 567 | "admin_state_up of router to True") |
Yair Fried | 413bf2d | 2014-11-19 17:07:11 +0200 | [diff] [blame] | 568 | |
Ken'ichi Ohmichi | c85a951 | 2017-01-27 18:34:24 -0800 | [diff] [blame] | 569 | @decorators.idempotent_id('d8bb918e-e2df-48b2-97cd-b73c95450980') |
Thiago Paiva | 66cded2 | 2016-08-15 14:55:58 -0300 | [diff] [blame] | 570 | @testtools.skipIf(CONF.network.shared_physical_network, |
| 571 | 'network isolation not available') |
Yair Fried | 413bf2d | 2014-11-19 17:07:11 +0200 | [diff] [blame] | 572 | @testtools.skipUnless(CONF.scenario.dhcp_client, |
| 573 | "DHCP client is not available.") |
Jordan Pittier | 3b46d27 | 2017-04-12 16:17:28 +0200 | [diff] [blame] | 574 | @decorators.attr(type='slow') |
Andrea Frittoli | cd36841 | 2017-08-14 21:37:56 +0100 | [diff] [blame] | 575 | @utils.services('compute', 'network') |
Yair Fried | 413bf2d | 2014-11-19 17:07:11 +0200 | [diff] [blame] | 576 | def test_subnet_details(self): |
Ken'ichi Ohmichi | 2e2ee19 | 2015-11-19 09:48:27 +0000 | [diff] [blame] | 577 | """Tests that subnet's extra configuration details are affecting VMs. |
| 578 | |
Sergey Vilgelm | eac094a | 2018-11-21 18:27:51 -0600 | [diff] [blame] | 579 | This test relies on non-shared, isolated tenant networks. |
Yair Fried | 413bf2d | 2014-11-19 17:07:11 +0200 | [diff] [blame] | 580 | |
Sergey Vilgelm | eac094a | 2018-11-21 18:27:51 -0600 | [diff] [blame] | 581 | NOTE: Neutron subnets push data to servers via dhcp-agent, so any |
| 582 | update in subnet requires server to actively renew its DHCP lease. |
Yair Fried | 413bf2d | 2014-11-19 17:07:11 +0200 | [diff] [blame] | 583 | |
Sergey Vilgelm | eac094a | 2018-11-21 18:27:51 -0600 | [diff] [blame] | 584 | 1. Configure subnet with dns nameserver |
| 585 | 2. retrieve the VM's configured dns and verify it matches the one |
| 586 | configured for the subnet. |
| 587 | 3. update subnet's dns |
| 588 | 4. retrieve the VM's configured dns and verify it matches the new one |
| 589 | configured for the subnet. |
Yair Fried | 413bf2d | 2014-11-19 17:07:11 +0200 | [diff] [blame] | 590 | |
Sergey Vilgelm | eac094a | 2018-11-21 18:27:51 -0600 | [diff] [blame] | 591 | TODO(yfried): add host_routes |
Yair Fried | 413bf2d | 2014-11-19 17:07:11 +0200 | [diff] [blame] | 592 | |
Sergey Vilgelm | eac094a | 2018-11-21 18:27:51 -0600 | [diff] [blame] | 593 | any resolution check would be testing either: |
| 594 | |
| 595 | * l3 forwarding (tested in test_network_basic_ops) |
| 596 | * Name resolution of an external DNS nameserver - out of scope for |
| 597 | Tempest |
Yair Fried | 413bf2d | 2014-11-19 17:07:11 +0200 | [diff] [blame] | 598 | """ |
| 599 | # this test check only updates (no actual resolution) so using |
| 600 | # arbitrary ip addresses as nameservers, instead of parsing CONF |
| 601 | initial_dns_server = '1.2.3.4' |
| 602 | alt_dns_server = '9.8.7.6' |
Yair Fried | bb0ea39 | 2015-01-19 07:26:08 +0000 | [diff] [blame] | 603 | |
Bence Romsics | 199e022 | 2019-01-28 14:18:00 +0100 | [diff] [blame] | 604 | # Original timeouts are suggested by salvatore-orlando in |
Yair Fried | bb0ea39 | 2015-01-19 07:26:08 +0000 | [diff] [blame] | 605 | # https://bugs.launchpad.net/neutron/+bug/1412325/comments/3 |
Bence Romsics | 199e022 | 2019-01-28 14:18:00 +0100 | [diff] [blame] | 606 | # |
| 607 | # Compared to that renew_delay was increased, because |
| 608 | # busybox's udhcpc accepts SIGUSR1 as a renew request. Internally |
| 609 | # it goes into RENEW_REQUESTED state. If it receives a 2nd SIGUSR1 |
| 610 | # signal while in that state then it calls the deconfig script |
| 611 | # ("/sbin/cirros-dhcpc deconfig" in sufficiently new cirros versions) |
| 612 | # which leads to the address being transiently deconfigured which |
| 613 | # for our case is unwanted. |
| 614 | renew_delay = 3 * CONF.network.build_interval |
Yair Fried | bb0ea39 | 2015-01-19 07:26:08 +0000 | [diff] [blame] | 615 | renew_timeout = CONF.network.build_timeout |
| 616 | |
Yair Fried | 413bf2d | 2014-11-19 17:07:11 +0200 | [diff] [blame] | 617 | self._setup_network_and_servers(dns_nameservers=[initial_dns_server]) |
zhufl | 0ec74c4 | 2017-11-15 14:02:28 +0800 | [diff] [blame] | 618 | self._check_public_network_connectivity(should_connect=True) |
Yair Fried | 413bf2d | 2014-11-19 17:07:11 +0200 | [diff] [blame] | 619 | |
| 620 | floating_ip, server = self.floating_ip_tuple |
Steve Heyman | 33735f2 | 2016-05-24 09:28:08 -0500 | [diff] [blame] | 621 | ip_address = floating_ip['floating_ip_address'] |
Yair Fried | 413bf2d | 2014-11-19 17:07:11 +0200 | [diff] [blame] | 622 | private_key = self._get_server_key(server) |
Jordan Pittier | bbb1712 | 2016-01-26 17:10:55 +0100 | [diff] [blame] | 623 | ssh_client = self.get_remote_client( |
zhufl | f52c759 | 2017-05-25 13:55:24 +0800 | [diff] [blame] | 624 | ip_address, private_key=private_key, server=server) |
Yair Fried | 413bf2d | 2014-11-19 17:07:11 +0200 | [diff] [blame] | 625 | |
armando-migliaccio | 424aa41 | 2015-02-22 17:55:17 -0800 | [diff] [blame] | 626 | dns_servers = [initial_dns_server] |
| 627 | servers = ssh_client.get_dns_servers() |
| 628 | self.assertEqual(set(dns_servers), set(servers), |
| 629 | 'Looking for servers: {trgt_serv}. ' |
| 630 | 'Retrieved DNS nameservers: {act_serv} ' |
| 631 | 'From host: {host}.' |
| 632 | .format(host=ssh_client.ssh_client.host, |
| 633 | act_serv=servers, |
| 634 | trgt_serv=dns_servers)) |
Yair Fried | 413bf2d | 2014-11-19 17:07:11 +0200 | [diff] [blame] | 635 | |
Steve Heyman | 33735f2 | 2016-05-24 09:28:08 -0500 | [diff] [blame] | 636 | self.subnet = self.subnets_client.update_subnet( |
| 637 | self.subnet['id'], dns_nameservers=[alt_dns_server])['subnet'] |
| 638 | |
Yair Fried | 413bf2d | 2014-11-19 17:07:11 +0200 | [diff] [blame] | 639 | # asserts that Neutron DB has updated the nameservers |
Steve Heyman | 33735f2 | 2016-05-24 09:28:08 -0500 | [diff] [blame] | 640 | self.assertEqual([alt_dns_server], self.subnet['dns_nameservers'], |
Yair Fried | 413bf2d | 2014-11-19 17:07:11 +0200 | [diff] [blame] | 641 | "Failed to update subnet's nameservers") |
| 642 | |
Yair Fried | bb0ea39 | 2015-01-19 07:26:08 +0000 | [diff] [blame] | 643 | def check_new_dns_server(): |
Ken'ichi Ohmichi | 2e2ee19 | 2015-11-19 09:48:27 +0000 | [diff] [blame] | 644 | # NOTE: Server needs to renew its dhcp lease in order to get new |
| 645 | # definitions from subnet |
| 646 | # NOTE(amuller): we are renewing the lease as part of the retry |
| 647 | # because Neutron updates dnsmasq asynchronously after the |
| 648 | # subnet-update API call returns. |
Ken'ichi Ohmichi | 4e33785 | 2017-03-01 12:04:23 -0800 | [diff] [blame] | 649 | ssh_client.renew_lease(fixed_ip=floating_ip['fixed_ip_address'], |
| 650 | dhcp_client=CONF.scenario.dhcp_client) |
armando-migliaccio | 424aa41 | 2015-02-22 17:55:17 -0800 | [diff] [blame] | 651 | if ssh_client.get_dns_servers() != [alt_dns_server]: |
Yair Fried | bb0ea39 | 2015-01-19 07:26:08 +0000 | [diff] [blame] | 652 | LOG.debug("Failed to update DNS nameservers") |
| 653 | return False |
| 654 | return True |
| 655 | |
Jordan Pittier | 35a6375 | 2016-08-30 13:09:12 +0200 | [diff] [blame] | 656 | self.assertTrue(test_utils.call_until_true(check_new_dns_server, |
| 657 | renew_timeout, |
| 658 | renew_delay), |
Yair Fried | bb0ea39 | 2015-01-19 07:26:08 +0000 | [diff] [blame] | 659 | msg="DHCP renewal failed to fetch " |
| 660 | "new DNS nameservers") |
Thalabathy Venkatesan | 76553d8 | 2015-01-07 22:54:39 -0800 | [diff] [blame] | 661 | |
Ken'ichi Ohmichi | c85a951 | 2017-01-27 18:34:24 -0800 | [diff] [blame] | 662 | @decorators.idempotent_id('f5dfcc22-45fd-409f-954c-5bd500d7890b') |
Itzik Brown | beb30d3 | 2015-03-29 09:42:54 +0300 | [diff] [blame] | 663 | @testtools.skipUnless(CONF.network_feature_enabled.port_admin_state_change, |
| 664 | "Changing a port's admin state is not supported " |
| 665 | "by the test environment") |
Jordan Pittier | 3b46d27 | 2017-04-12 16:17:28 +0200 | [diff] [blame] | 666 | @decorators.attr(type='slow') |
Andrea Frittoli | cd36841 | 2017-08-14 21:37:56 +0100 | [diff] [blame] | 667 | @utils.services('compute', 'network') |
Thalabathy Venkatesan | 76553d8 | 2015-01-07 22:54:39 -0800 | [diff] [blame] | 668 | def test_update_instance_port_admin_state(self): |
Ken'ichi Ohmichi | 2e2ee19 | 2015-11-19 09:48:27 +0000 | [diff] [blame] | 669 | """Test to update admin_state_up attribute of instance port |
| 670 | |
Carlos Goncalves | 539f836 | 2016-11-04 14:55:02 +0100 | [diff] [blame] | 671 | 1. Check public and project connectivity before updating |
Thalabathy Venkatesan | 76553d8 | 2015-01-07 22:54:39 -0800 | [diff] [blame] | 672 | admin_state_up attribute of instance port to False |
Carlos Goncalves | 539f836 | 2016-11-04 14:55:02 +0100 | [diff] [blame] | 673 | 2. Check public and project connectivity after updating |
Thalabathy Venkatesan | 76553d8 | 2015-01-07 22:54:39 -0800 | [diff] [blame] | 674 | admin_state_up attribute of instance port to False |
Carlos Goncalves | 539f836 | 2016-11-04 14:55:02 +0100 | [diff] [blame] | 675 | 3. Check public and project connectivity after updating |
Thalabathy Venkatesan | 76553d8 | 2015-01-07 22:54:39 -0800 | [diff] [blame] | 676 | admin_state_up attribute of instance port to True |
| 677 | """ |
| 678 | self._setup_network_and_servers() |
Ferenc Horváth | bce1fcf | 2017-06-07 11:19:51 +0200 | [diff] [blame] | 679 | _, server = self.floating_ip_tuple |
Thalabathy Venkatesan | 76553d8 | 2015-01-07 22:54:39 -0800 | [diff] [blame] | 680 | server_id = server['id'] |
jeremy.zhang | 5870ff1 | 2017-05-25 11:24:23 +0800 | [diff] [blame] | 681 | port_id = self.os_admin.ports_client.list_ports( |
Jordan Pittier | 64e6b44 | 2017-02-20 19:29:02 +0100 | [diff] [blame] | 682 | device_id=server_id)['ports'][0]['id'] |
Carlos Goncalves | 539f836 | 2016-11-04 14:55:02 +0100 | [diff] [blame] | 683 | server_pip = server['addresses'][self.network['name']][0]['addr'] |
| 684 | |
| 685 | server2 = self._create_server(self.network) |
| 686 | server2_fip = self.create_floating_ip(server2) |
| 687 | |
| 688 | private_key = self._get_server_key(server2) |
| 689 | ssh_client = self.get_remote_client(server2_fip['floating_ip_address'], |
zhufl | f52c759 | 2017-05-25 13:55:24 +0800 | [diff] [blame] | 690 | private_key=private_key, |
| 691 | server=server2) |
Carlos Goncalves | 539f836 | 2016-11-04 14:55:02 +0100 | [diff] [blame] | 692 | |
zhufl | 0ec74c4 | 2017-11-15 14:02:28 +0800 | [diff] [blame] | 693 | self._check_public_network_connectivity( |
Thalabathy Venkatesan | 76553d8 | 2015-01-07 22:54:39 -0800 | [diff] [blame] | 694 | should_connect=True, msg="before updating " |
| 695 | "admin_state_up of instance port to False") |
YAMAMOTO Takashi | 4c3ebb0 | 2017-01-25 16:04:30 +0900 | [diff] [blame] | 696 | self.check_remote_connectivity(ssh_client, dest=server_pip, |
| 697 | should_succeed=True) |
John Warren | 49c0fe5 | 2015-10-22 12:35:54 -0400 | [diff] [blame] | 698 | self.ports_client.update_port(port_id, admin_state_up=False) |
zhufl | 0ec74c4 | 2017-11-15 14:02:28 +0800 | [diff] [blame] | 699 | self._check_public_network_connectivity( |
Thalabathy Venkatesan | 76553d8 | 2015-01-07 22:54:39 -0800 | [diff] [blame] | 700 | should_connect=False, msg="after updating " |
| 701 | "admin_state_up of instance port to False", |
| 702 | should_check_floating_ip_status=False) |
YAMAMOTO Takashi | 4c3ebb0 | 2017-01-25 16:04:30 +0900 | [diff] [blame] | 703 | self.check_remote_connectivity(ssh_client, dest=server_pip, |
| 704 | should_succeed=False) |
John Warren | 49c0fe5 | 2015-10-22 12:35:54 -0400 | [diff] [blame] | 705 | self.ports_client.update_port(port_id, admin_state_up=True) |
zhufl | 0ec74c4 | 2017-11-15 14:02:28 +0800 | [diff] [blame] | 706 | self._check_public_network_connectivity( |
Thalabathy Venkatesan | 76553d8 | 2015-01-07 22:54:39 -0800 | [diff] [blame] | 707 | should_connect=True, msg="after updating " |
| 708 | "admin_state_up of instance port to True") |
YAMAMOTO Takashi | 4c3ebb0 | 2017-01-25 16:04:30 +0900 | [diff] [blame] | 709 | self.check_remote_connectivity(ssh_client, dest=server_pip, |
| 710 | should_succeed=True) |
Matt Riedemann | 1794073 | 2015-03-13 14:18:19 +0000 | [diff] [blame] | 711 | |
Ken'ichi Ohmichi | c85a951 | 2017-01-27 18:34:24 -0800 | [diff] [blame] | 712 | @decorators.idempotent_id('759462e1-8535-46b0-ab3a-33aa45c55aaa') |
Jordan Pittier | 3b46d27 | 2017-04-12 16:17:28 +0200 | [diff] [blame] | 713 | @decorators.attr(type='slow') |
Andrea Frittoli | cd36841 | 2017-08-14 21:37:56 +0100 | [diff] [blame] | 714 | @utils.services('compute', 'network') |
Matt Riedemann | 1794073 | 2015-03-13 14:18:19 +0000 | [diff] [blame] | 715 | def test_preserve_preexisting_port(self): |
Ken'ichi Ohmichi | 2e2ee19 | 2015-11-19 09:48:27 +0000 | [diff] [blame] | 716 | """Test preserve pre-existing port |
| 717 | |
| 718 | Tests that a pre-existing port provided on server boot is not deleted |
| 719 | if the server is deleted. |
Matt Riedemann | 1794073 | 2015-03-13 14:18:19 +0000 | [diff] [blame] | 720 | |
| 721 | Nova should unbind the port from the instance on delete if the port was |
| 722 | not created by Nova as part of the boot request. |
Matt Riedemann | a8c641a | 2016-07-12 17:07:33 -0400 | [diff] [blame] | 723 | |
| 724 | We should also be able to boot another server with the same port. |
Matt Riedemann | 1794073 | 2015-03-13 14:18:19 +0000 | [diff] [blame] | 725 | """ |
| 726 | # Setup the network, create a port and boot the server from that port. |
| 727 | self._setup_network_and_servers(boot_with_port=True) |
| 728 | _, server = self.floating_ip_tuple |
Shuquan Huang | b5c8beb | 2015-08-05 14:14:01 +0000 | [diff] [blame] | 729 | self.assertEqual(1, len(self.ports), |
| 730 | 'There should only be one port created for ' |
| 731 | 'server %s.' % server['id']) |
| 732 | port_id = self.ports[0]['port'] |
| 733 | self.assertIsNotNone(port_id, |
Matt Riedemann | 1794073 | 2015-03-13 14:18:19 +0000 | [diff] [blame] | 734 | 'Server should have been created from a ' |
| 735 | 'pre-existing port.') |
| 736 | # Assert the port is bound to the server. |
jeremy.zhang | 5870ff1 | 2017-05-25 11:24:23 +0800 | [diff] [blame] | 737 | port_list = self.os_admin.ports_client.list_ports( |
Jordan Pittier | 64e6b44 | 2017-02-20 19:29:02 +0100 | [diff] [blame] | 738 | device_id=server['id'], network_id=self.network['id'])['ports'] |
Matt Riedemann | 1794073 | 2015-03-13 14:18:19 +0000 | [diff] [blame] | 739 | self.assertEqual(1, len(port_list), |
| 740 | 'There should only be one port created for ' |
| 741 | 'server %s.' % server['id']) |
Shuquan Huang | b5c8beb | 2015-08-05 14:14:01 +0000 | [diff] [blame] | 742 | self.assertEqual(port_id, port_list[0]['id']) |
Matt Riedemann | 1794073 | 2015-03-13 14:18:19 +0000 | [diff] [blame] | 743 | # Delete the server. |
| 744 | self.servers_client.delete_server(server['id']) |
Ken'ichi Ohmichi | e91a0c6 | 2015-08-13 02:09:16 +0000 | [diff] [blame] | 745 | waiters.wait_for_server_termination(self.servers_client, server['id']) |
Matt Riedemann | 1794073 | 2015-03-13 14:18:19 +0000 | [diff] [blame] | 746 | # Assert the port still exists on the network but is unbound from |
| 747 | # the deleted server. |
John Warren | 49c0fe5 | 2015-10-22 12:35:54 -0400 | [diff] [blame] | 748 | port = self.ports_client.show_port(port_id)['port'] |
Matt Riedemann | 1794073 | 2015-03-13 14:18:19 +0000 | [diff] [blame] | 749 | self.assertEqual(self.network['id'], port['network_id']) |
| 750 | self.assertEqual('', port['device_id']) |
| 751 | self.assertEqual('', port['device_owner']) |
Yair Fried | 564d89d | 2015-08-06 17:02:12 +0300 | [diff] [blame] | 752 | |
Matt Riedemann | a8c641a | 2016-07-12 17:07:33 -0400 | [diff] [blame] | 753 | # Boot another server with the same port to make sure nothing was |
| 754 | # left around that could cause issues. |
zhufl | 24208c2 | 2016-10-25 15:23:48 +0800 | [diff] [blame] | 755 | server = self._create_server(self.network, port['id']) |
jeremy.zhang | 5870ff1 | 2017-05-25 11:24:23 +0800 | [diff] [blame] | 756 | port_list = self.os_admin.ports_client.list_ports( |
Jordan Pittier | 64e6b44 | 2017-02-20 19:29:02 +0100 | [diff] [blame] | 757 | device_id=server['id'], network_id=self.network['id'])['ports'] |
Matt Riedemann | a8c641a | 2016-07-12 17:07:33 -0400 | [diff] [blame] | 758 | self.assertEqual(1, len(port_list), |
| 759 | 'There should only be one port created for ' |
| 760 | 'server %s.' % server['id']) |
| 761 | self.assertEqual(port['id'], port_list[0]['id']) |
| 762 | |
Andrea Frittoli | cd36841 | 2017-08-14 21:37:56 +0100 | [diff] [blame] | 763 | @utils.requires_ext(service='network', extension='l3_agent_scheduler') |
Ken'ichi Ohmichi | c85a951 | 2017-01-27 18:34:24 -0800 | [diff] [blame] | 764 | @decorators.idempotent_id('2e788c46-fb3f-4ac9-8f82-0561555bea73') |
Jordan Pittier | 3b46d27 | 2017-04-12 16:17:28 +0200 | [diff] [blame] | 765 | @decorators.attr(type='slow') |
Andrea Frittoli | cd36841 | 2017-08-14 21:37:56 +0100 | [diff] [blame] | 766 | @utils.services('compute', 'network') |
Yair Fried | 564d89d | 2015-08-06 17:02:12 +0300 | [diff] [blame] | 767 | def test_router_rescheduling(self): |
| 768 | """Tests that router can be removed from agent and add to a new agent. |
| 769 | |
| 770 | 1. Verify connectivity |
| 771 | 2. Remove router from all l3-agents |
| 772 | 3. Verify connectivity is down |
| 773 | 4. Assign router to new l3-agent (or old one if no new agent is |
Sergey Vilgelm | eac094a | 2018-11-21 18:27:51 -0600 | [diff] [blame] | 774 | available) |
Yair Fried | 564d89d | 2015-08-06 17:02:12 +0300 | [diff] [blame] | 775 | 5. Verify connectivity |
| 776 | """ |
| 777 | |
| 778 | # TODO(yfried): refactor this test to be used for other agents (dhcp) |
| 779 | # as well |
| 780 | |
jeremy.zhang | 5870ff1 | 2017-05-25 11:24:23 +0800 | [diff] [blame] | 781 | list_hosts = (self.os_admin.routers_client. |
Yair Fried | 564d89d | 2015-08-06 17:02:12 +0300 | [diff] [blame] | 782 | list_l3_agents_hosting_router) |
jeremy.zhang | 5870ff1 | 2017-05-25 11:24:23 +0800 | [diff] [blame] | 783 | schedule_router = (self.os_admin.network_agents_client. |
Ken'ichi Ohmichi | 70a24e9 | 2016-01-06 01:08:27 +0000 | [diff] [blame] | 784 | create_router_on_l3_agent) |
jeremy.zhang | 5870ff1 | 2017-05-25 11:24:23 +0800 | [diff] [blame] | 785 | unschedule_router = (self.os_admin.network_agents_client. |
Ken'ichi Ohmichi | 70a24e9 | 2016-01-06 01:08:27 +0000 | [diff] [blame] | 786 | delete_router_from_l3_agent) |
Yair Fried | 564d89d | 2015-08-06 17:02:12 +0300 | [diff] [blame] | 787 | |
Jordan Pittier | 64e6b44 | 2017-02-20 19:29:02 +0100 | [diff] [blame] | 788 | agent_list_alive = set( |
| 789 | a["id"] for a in |
jeremy.zhang | 5870ff1 | 2017-05-25 11:24:23 +0800 | [diff] [blame] | 790 | self.os_admin.network_agents_client.list_agents( |
Jordan Pittier | 64e6b44 | 2017-02-20 19:29:02 +0100 | [diff] [blame] | 791 | agent_type="L3 agent")['agents'] if a["alive"] is True |
| 792 | ) |
Yair Fried | 564d89d | 2015-08-06 17:02:12 +0300 | [diff] [blame] | 793 | self._setup_network_and_servers() |
Kevin Benton | 3b63aa1 | 2015-08-27 17:15:44 -0700 | [diff] [blame] | 794 | |
| 795 | # NOTE(kevinbenton): we have to use the admin credentials to check |
Sean Dague | ed6e586 | 2016-04-04 10:49:13 -0400 | [diff] [blame] | 796 | # for the distributed flag because self.router only has a project view. |
jeremy.zhang | 5870ff1 | 2017-05-25 11:24:23 +0800 | [diff] [blame] | 797 | admin = self.os_admin.routers_client.show_router( |
Steve Heyman | 33735f2 | 2016-05-24 09:28:08 -0500 | [diff] [blame] | 798 | self.router['id']) |
Kevin Benton | 3b63aa1 | 2015-08-27 17:15:44 -0700 | [diff] [blame] | 799 | if admin['router'].get('distributed', False): |
| 800 | msg = "Rescheduling test does not apply to distributed routers." |
| 801 | raise self.skipException(msg) |
| 802 | |
zhufl | 0ec74c4 | 2017-11-15 14:02:28 +0800 | [diff] [blame] | 803 | self._check_public_network_connectivity(should_connect=True) |
Yair Fried | 564d89d | 2015-08-06 17:02:12 +0300 | [diff] [blame] | 804 | |
| 805 | # remove resource from agents |
| 806 | hosting_agents = set(a["id"] for a in |
Steve Heyman | 33735f2 | 2016-05-24 09:28:08 -0500 | [diff] [blame] | 807 | list_hosts(self.router['id'])['agents']) |
shipeiqi | c9a4a10 | 2016-04-20 17:39:03 +0800 | [diff] [blame] | 808 | no_migration = agent_list_alive == hosting_agents |
Yair Fried | 564d89d | 2015-08-06 17:02:12 +0300 | [diff] [blame] | 809 | LOG.info("Router will be assigned to {mig} hosting agent". |
| 810 | format(mig="the same" if no_migration else "a new")) |
| 811 | |
| 812 | for hosting_agent in hosting_agents: |
Steve Heyman | 33735f2 | 2016-05-24 09:28:08 -0500 | [diff] [blame] | 813 | unschedule_router(hosting_agent, self.router['id']) |
Yair Fried | 564d89d | 2015-08-06 17:02:12 +0300 | [diff] [blame] | 814 | self.assertNotIn(hosting_agent, |
| 815 | [a["id"] for a in |
Steve Heyman | 33735f2 | 2016-05-24 09:28:08 -0500 | [diff] [blame] | 816 | list_hosts(self.router['id'])['agents']], |
Yair Fried | 564d89d | 2015-08-06 17:02:12 +0300 | [diff] [blame] | 817 | 'unscheduling router failed') |
| 818 | |
| 819 | # verify resource is un-functional |
zhufl | 0ec74c4 | 2017-11-15 14:02:28 +0800 | [diff] [blame] | 820 | self._check_public_network_connectivity( |
Yair Fried | 564d89d | 2015-08-06 17:02:12 +0300 | [diff] [blame] | 821 | should_connect=False, |
| 822 | msg='after router unscheduling', |
Yair Fried | 564d89d | 2015-08-06 17:02:12 +0300 | [diff] [blame] | 823 | ) |
| 824 | |
| 825 | # schedule resource to new agent |
| 826 | target_agent = list(hosting_agents if no_migration else |
shipeiqi | c9a4a10 | 2016-04-20 17:39:03 +0800 | [diff] [blame] | 827 | agent_list_alive - hosting_agents)[0] |
Yair Fried | 564d89d | 2015-08-06 17:02:12 +0300 | [diff] [blame] | 828 | schedule_router(target_agent, |
piyush110786 | bc0faa6 | 2015-12-03 14:54:48 +0530 | [diff] [blame] | 829 | router_id=self.router['id']) |
Yair Fried | 564d89d | 2015-08-06 17:02:12 +0300 | [diff] [blame] | 830 | self.assertEqual( |
| 831 | target_agent, |
Steve Heyman | 33735f2 | 2016-05-24 09:28:08 -0500 | [diff] [blame] | 832 | list_hosts(self.router['id'])['agents'][0]['id'], |
Yair Fried | 564d89d | 2015-08-06 17:02:12 +0300 | [diff] [blame] | 833 | "Router failed to reschedule. Hosting agent doesn't match " |
| 834 | "target agent") |
| 835 | |
| 836 | # verify resource is functional |
zhufl | 0ec74c4 | 2017-11-15 14:02:28 +0800 | [diff] [blame] | 837 | self._check_public_network_connectivity( |
Yair Fried | 564d89d | 2015-08-06 17:02:12 +0300 | [diff] [blame] | 838 | should_connect=True, |
| 839 | msg='After router rescheduling') |
Yair Fried | bc46f59 | 2015-11-18 16:29:34 +0200 | [diff] [blame] | 840 | |
Andrea Frittoli | cd36841 | 2017-08-14 21:37:56 +0100 | [diff] [blame] | 841 | @utils.requires_ext(service='network', extension='port-security') |
Evgeny Antyshev | aaf3fc9 | 2016-02-15 11:49:22 +0000 | [diff] [blame] | 842 | @testtools.skipUnless(CONF.compute_feature_enabled.interface_attach, |
| 843 | 'NIC hotplug not available') |
Ken'ichi Ohmichi | c85a951 | 2017-01-27 18:34:24 -0800 | [diff] [blame] | 844 | @decorators.idempotent_id('7c0bb1a2-d053-49a4-98f9-ca1a1d849f63') |
Jordan Pittier | 3b46d27 | 2017-04-12 16:17:28 +0200 | [diff] [blame] | 845 | @decorators.attr(type='slow') |
Andrea Frittoli | cd36841 | 2017-08-14 21:37:56 +0100 | [diff] [blame] | 846 | @utils.services('compute', 'network') |
Yair Fried | bc46f59 | 2015-11-18 16:29:34 +0200 | [diff] [blame] | 847 | def test_port_security_macspoofing_port(self): |
| 848 | """Tests port_security extension enforces mac spoofing |
| 849 | |
Eran Kuris | 7969e80 | 2015-12-15 13:43:46 +0200 | [diff] [blame] | 850 | Neutron security groups always apply anti-spoof rules on the VMs. This |
| 851 | allows traffic to originate and terminate at the VM as expected, but |
| 852 | prevents traffic to pass through the VM. Anti-spoof rules are not |
| 853 | required in cases where the VM routes traffic through it. |
| 854 | |
Sergey Vilgelm | eac094a | 2018-11-21 18:27:51 -0600 | [diff] [blame] | 855 | The test steps are: |
| 856 | |
Eran Kuris | 7969e80 | 2015-12-15 13:43:46 +0200 | [diff] [blame] | 857 | 1. Create a new network. |
| 858 | 2. Connect (hotplug) the VM to a new network. |
YAMAMOTO Takashi | c397841 | 2015-11-30 23:55:56 +0900 | [diff] [blame] | 859 | 3. Check the VM can ping a server on the new network ("peer") |
Eran Kuris | 7969e80 | 2015-12-15 13:43:46 +0200 | [diff] [blame] | 860 | 4. Spoof the mac address of the new VM interface. |
| 861 | 5. Check the Security Group enforces mac spoofing and blocks pings via |
YAMAMOTO Takashi | c397841 | 2015-11-30 23:55:56 +0900 | [diff] [blame] | 862 | spoofed interface (VM cannot ping the peer). |
Eran Kuris | 7969e80 | 2015-12-15 13:43:46 +0200 | [diff] [blame] | 863 | 6. Disable port-security of the spoofed port- set the flag to false. |
| 864 | 7. Retest 3rd step and check that the Security Group allows pings via |
Sergey Vilgelm | eac094a | 2018-11-21 18:27:51 -0600 | [diff] [blame] | 865 | the spoofed interface. |
Yair Fried | bc46f59 | 2015-11-18 16:29:34 +0200 | [diff] [blame] | 866 | """ |
Eran Kuris | 7969e80 | 2015-12-15 13:43:46 +0200 | [diff] [blame] | 867 | |
Yair Fried | bc46f59 | 2015-11-18 16:29:34 +0200 | [diff] [blame] | 868 | spoof_mac = "00:00:00:00:00:01" |
| 869 | |
| 870 | # Create server |
| 871 | self._setup_network_and_servers() |
zhufl | 0ec74c4 | 2017-11-15 14:02:28 +0800 | [diff] [blame] | 872 | self._check_public_network_connectivity(should_connect=True) |
Yair Fried | bc46f59 | 2015-11-18 16:29:34 +0200 | [diff] [blame] | 873 | self._create_new_network() |
| 874 | self._hotplug_server() |
| 875 | fip, server = self.floating_ip_tuple |
jeremy.zhang | 5870ff1 | 2017-05-25 11:24:23 +0800 | [diff] [blame] | 876 | new_ports = self.os_admin.ports_client.list_ports( |
Jordan Pittier | 64e6b44 | 2017-02-20 19:29:02 +0100 | [diff] [blame] | 877 | device_id=server["id"], network_id=self.new_net["id"])['ports'] |
Yair Fried | bc46f59 | 2015-11-18 16:29:34 +0200 | [diff] [blame] | 878 | spoof_port = new_ports[0] |
| 879 | private_key = self._get_server_key(server) |
Steve Heyman | 33735f2 | 2016-05-24 09:28:08 -0500 | [diff] [blame] | 880 | ssh_client = self.get_remote_client(fip['floating_ip_address'], |
zhufl | f52c759 | 2017-05-25 13:55:24 +0800 | [diff] [blame] | 881 | private_key=private_key, |
| 882 | server=server) |
Evgeny Antyshev | 9b77ae5 | 2016-02-16 09:48:57 +0000 | [diff] [blame] | 883 | spoof_nic = ssh_client.get_nic_name_by_mac(spoof_port["mac_address"]) |
zhufl | 24208c2 | 2016-10-25 15:23:48 +0800 | [diff] [blame] | 884 | peer = self._create_server(self.new_net) |
Steve Heyman | 33735f2 | 2016-05-24 09:28:08 -0500 | [diff] [blame] | 885 | peer_address = peer['addresses'][self.new_net['name']][0]['addr'] |
YAMAMOTO Takashi | 4c3ebb0 | 2017-01-25 16:04:30 +0900 | [diff] [blame] | 886 | self.check_remote_connectivity(ssh_client, dest=peer_address, |
| 887 | nic=spoof_nic, should_succeed=True) |
Ken'ichi Ohmichi | 5129c62 | 2017-03-22 11:18:42 -0700 | [diff] [blame] | 888 | # Set a mac address by making nic down temporary |
| 889 | cmd = ("sudo ip link set {nic} down;" |
| 890 | "sudo ip link set dev {nic} address {mac};" |
| 891 | "sudo ip link set {nic} up").format(nic=spoof_nic, |
| 892 | mac=spoof_mac) |
| 893 | ssh_client.exec_command(cmd) |
| 894 | |
Yair Fried | bc46f59 | 2015-11-18 16:29:34 +0200 | [diff] [blame] | 895 | new_mac = ssh_client.get_mac_address(nic=spoof_nic) |
| 896 | self.assertEqual(spoof_mac, new_mac) |
YAMAMOTO Takashi | 4c3ebb0 | 2017-01-25 16:04:30 +0900 | [diff] [blame] | 897 | self.check_remote_connectivity(ssh_client, dest=peer_address, |
| 898 | nic=spoof_nic, should_succeed=False) |
Yair Fried | bc46f59 | 2015-11-18 16:29:34 +0200 | [diff] [blame] | 899 | self.ports_client.update_port(spoof_port["id"], |
| 900 | port_security_enabled=False, |
| 901 | security_groups=[]) |
YAMAMOTO Takashi | 4c3ebb0 | 2017-01-25 16:04:30 +0900 | [diff] [blame] | 902 | self.check_remote_connectivity(ssh_client, dest=peer_address, |
| 903 | nic=spoof_nic, should_succeed=True) |