| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 1 | # Copyright 2013 OpenStack Foundation |
| 2 | # All Rights Reserved. |
| 3 | # |
| 4 | # Licensed under the Apache License, Version 2.0 (the "License"); you may |
| 5 | # not use this file except in compliance with the License. You may obtain |
| 6 | # a copy of the License at |
| 7 | # |
| 8 | # http://www.apache.org/licenses/LICENSE-2.0 |
| 9 | # |
| 10 | # Unless required by applicable law or agreed to in writing, software |
| 11 | # distributed under the License is distributed on an "AS IS" BASIS, WITHOUT |
| 12 | # WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the |
| 13 | # License for the specific language governing permissions and limitations |
| 14 | # under the License. |
| 15 | |
| ChenZheng | 04ac694 | 2013-12-09 16:42:52 +0800 | [diff] [blame] | 16 | import netaddr |
| 17 | |
| rossella | e02431e | 2013-11-15 17:58:29 +0100 | [diff] [blame] | 18 | from tempest.api.network import base_routers as base |
| Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 19 | from tempest.common.utils import data_utils |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 20 | from tempest import config |
| armando-migliaccio | 89e5b96 | 2013-12-16 06:12:16 -0800 | [diff] [blame] | 21 | from tempest import test |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 22 | |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 23 | CONF = config.CONF |
| 24 | |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 25 | |
| rossella | e02431e | 2013-11-15 17:58:29 +0100 | [diff] [blame] | 26 | class RoutersTest(base.BaseRouterTest): |
| raiesmh08 | 6769832 | 2013-08-20 13:09:01 +0530 | [diff] [blame] | 27 | _interface = 'json' |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 28 | |
| 29 | @classmethod |
| 30 | def setUpClass(cls): |
| 31 | super(RoutersTest, cls).setUpClass() |
| 32 | |
| armando-migliaccio | 89e5b96 | 2013-12-16 06:12:16 -0800 | [diff] [blame] | 33 | @test.attr(type='smoke') |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 34 | def test_create_show_list_update_delete_router(self): |
| 35 | # Create a router |
| Salvatore Orlando | a85e8fe | 2013-09-20 03:48:02 -0700 | [diff] [blame] | 36 | # NOTE(salv-orlando): Do not invoke self.create_router |
| 37 | # as we need to check the response code |
| Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 38 | name = data_utils.rand_name('router-') |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 39 | resp, create_body = self.client.create_router( |
| 40 | name, external_gateway_info={ |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 41 | "network_id": CONF.network.public_network_id}, |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 42 | admin_state_up=False) |
| 43 | self.assertEqual('201', resp['status']) |
| 44 | self.addCleanup(self._delete_router, create_body['router']['id']) |
| 45 | self.assertEqual(create_body['router']['name'], name) |
| 46 | self.assertEqual( |
| 47 | create_body['router']['external_gateway_info']['network_id'], |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 48 | CONF.network.public_network_id) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 49 | self.assertEqual(create_body['router']['admin_state_up'], False) |
| 50 | # Show details of the created router |
| 51 | resp, show_body = self.client.show_router( |
| 52 | create_body['router']['id']) |
| 53 | self.assertEqual('200', resp['status']) |
| 54 | self.assertEqual(show_body['router']['name'], name) |
| 55 | self.assertEqual( |
| 56 | show_body['router']['external_gateway_info']['network_id'], |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 57 | CONF.network.public_network_id) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 58 | self.assertEqual(show_body['router']['admin_state_up'], False) |
| 59 | # List routers and verify if created router is there in response |
| 60 | resp, list_body = self.client.list_routers() |
| 61 | self.assertEqual('200', resp['status']) |
| 62 | routers_list = list() |
| 63 | for router in list_body['routers']: |
| 64 | routers_list.append(router['id']) |
| 65 | self.assertIn(create_body['router']['id'], routers_list) |
| 66 | # Update the name of router and verify if it is updated |
| 67 | updated_name = 'updated ' + name |
| 68 | resp, update_body = self.client.update_router( |
| 69 | create_body['router']['id'], name=updated_name) |
| 70 | self.assertEqual('200', resp['status']) |
| 71 | self.assertEqual(update_body['router']['name'], updated_name) |
| 72 | resp, show_body = self.client.show_router( |
| 73 | create_body['router']['id']) |
| 74 | self.assertEqual(show_body['router']['name'], updated_name) |
| 75 | |
| armando-migliaccio | 89e5b96 | 2013-12-16 06:12:16 -0800 | [diff] [blame] | 76 | @test.attr(type='smoke') |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 77 | def test_add_remove_router_interface_with_subnet_id(self): |
| 78 | network = self.create_network() |
| 79 | subnet = self.create_subnet(network) |
| Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 80 | router = self.create_router(data_utils.rand_name('router-')) |
| Chang Bo Guo | f099f80 | 2013-09-13 19:01:46 -0700 | [diff] [blame] | 81 | # Add router interface with subnet id |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 82 | resp, interface = self.client.add_router_interface_with_subnet_id( |
| Salvatore Orlando | a85e8fe | 2013-09-20 03:48:02 -0700 | [diff] [blame] | 83 | router['id'], subnet['id']) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 84 | self.assertEqual('200', resp['status']) |
| 85 | self.addCleanup(self._remove_router_interface_with_subnet_id, |
| Salvatore Orlando | a85e8fe | 2013-09-20 03:48:02 -0700 | [diff] [blame] | 86 | router['id'], subnet['id']) |
| Attila Fazekas | 4b8b008 | 2013-10-25 14:24:32 +0200 | [diff] [blame] | 87 | self.assertIn('subnet_id', interface.keys()) |
| 88 | self.assertIn('port_id', interface.keys()) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 89 | # Verify router id is equal to device id in port details |
| 90 | resp, show_port_body = self.client.show_port( |
| 91 | interface['port_id']) |
| 92 | self.assertEqual(show_port_body['port']['device_id'], |
| Salvatore Orlando | a85e8fe | 2013-09-20 03:48:02 -0700 | [diff] [blame] | 93 | router['id']) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 94 | |
| armando-migliaccio | 89e5b96 | 2013-12-16 06:12:16 -0800 | [diff] [blame] | 95 | @test.attr(type='smoke') |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 96 | def test_add_remove_router_interface_with_port_id(self): |
| 97 | network = self.create_network() |
| 98 | self.create_subnet(network) |
| Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 99 | router = self.create_router(data_utils.rand_name('router-')) |
| Eugene Nikanorov | e9d255a | 2013-12-18 16:31:42 +0400 | [diff] [blame] | 100 | resp, port_body = self.client.create_port( |
| 101 | network_id=network['id']) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 102 | # add router interface to port created above |
| 103 | resp, interface = self.client.add_router_interface_with_port_id( |
| Salvatore Orlando | a85e8fe | 2013-09-20 03:48:02 -0700 | [diff] [blame] | 104 | router['id'], port_body['port']['id']) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 105 | self.assertEqual('200', resp['status']) |
| 106 | self.addCleanup(self._remove_router_interface_with_port_id, |
| Salvatore Orlando | a85e8fe | 2013-09-20 03:48:02 -0700 | [diff] [blame] | 107 | router['id'], port_body['port']['id']) |
| Attila Fazekas | 4b8b008 | 2013-10-25 14:24:32 +0200 | [diff] [blame] | 108 | self.assertIn('subnet_id', interface.keys()) |
| 109 | self.assertIn('port_id', interface.keys()) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 110 | # Verify router id is equal to device id in port details |
| 111 | resp, show_port_body = self.client.show_port( |
| 112 | interface['port_id']) |
| 113 | self.assertEqual(show_port_body['port']['device_id'], |
| Salvatore Orlando | a85e8fe | 2013-09-20 03:48:02 -0700 | [diff] [blame] | 114 | router['id']) |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 115 | |
| 116 | def _verify_router_gateway(self, router_id, exp_ext_gw_info=None): |
| 117 | resp, show_body = self.client.show_router(router_id) |
| 118 | self.assertEqual('200', resp['status']) |
| 119 | actual_ext_gw_info = show_body['router']['external_gateway_info'] |
| 120 | if exp_ext_gw_info is None: |
| 121 | self.assertIsNone(actual_ext_gw_info) |
| 122 | return |
| 123 | # Verify only keys passed in exp_ext_gw_info |
| 124 | for k, v in exp_ext_gw_info.iteritems(): |
| 125 | self.assertEqual(v, actual_ext_gw_info[k]) |
| 126 | |
| 127 | def _verify_gateway_port(self, router_id): |
| 128 | resp, list_body = self.admin_client.list_ports( |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 129 | network_id=CONF.network.public_network_id, |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 130 | device_id=router_id) |
| 131 | self.assertEqual(len(list_body['ports']), 1) |
| 132 | gw_port = list_body['ports'][0] |
| 133 | fixed_ips = gw_port['fixed_ips'] |
| 134 | self.assertEqual(len(fixed_ips), 1) |
| 135 | resp, public_net_body = self.admin_client.show_network( |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 136 | CONF.network.public_network_id) |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 137 | public_subnet_id = public_net_body['network']['subnets'][0] |
| 138 | self.assertEqual(fixed_ips[0]['subnet_id'], public_subnet_id) |
| 139 | |
| armando-migliaccio | 89e5b96 | 2013-12-16 06:12:16 -0800 | [diff] [blame] | 140 | @test.attr(type='smoke') |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 141 | def test_update_router_set_gateway(self): |
| Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 142 | router = self.create_router(data_utils.rand_name('router-')) |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 143 | self.client.update_router( |
| 144 | router['id'], |
| 145 | external_gateway_info={ |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 146 | 'network_id': CONF.network.public_network_id}) |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 147 | # Verify operation - router |
| 148 | resp, show_body = self.client.show_router(router['id']) |
| 149 | self.assertEqual('200', resp['status']) |
| 150 | self._verify_router_gateway( |
| 151 | router['id'], |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 152 | {'network_id': CONF.network.public_network_id}) |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 153 | self._verify_gateway_port(router['id']) |
| 154 | |
| ronak | 79f1793 | 2014-01-27 17:56:21 -0800 | [diff] [blame] | 155 | @test.requires_ext(extension='ext-gw-mode', service='network') |
| armando-migliaccio | 89e5b96 | 2013-12-16 06:12:16 -0800 | [diff] [blame] | 156 | @test.attr(type='smoke') |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 157 | def test_update_router_set_gateway_with_snat_explicit(self): |
| Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 158 | router = self.create_router(data_utils.rand_name('router-')) |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 159 | self.admin_client.update_router_with_snat_gw_info( |
| 160 | router['id'], |
| 161 | external_gateway_info={ |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 162 | 'network_id': CONF.network.public_network_id, |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 163 | 'enable_snat': True}) |
| 164 | self._verify_router_gateway( |
| 165 | router['id'], |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 166 | {'network_id': CONF.network.public_network_id, |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 167 | 'enable_snat': True}) |
| 168 | self._verify_gateway_port(router['id']) |
| 169 | |
| ronak | 79f1793 | 2014-01-27 17:56:21 -0800 | [diff] [blame] | 170 | @test.requires_ext(extension='ext-gw-mode', service='network') |
| armando-migliaccio | 89e5b96 | 2013-12-16 06:12:16 -0800 | [diff] [blame] | 171 | @test.attr(type='smoke') |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 172 | def test_update_router_set_gateway_without_snat(self): |
| Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 173 | router = self.create_router(data_utils.rand_name('router-')) |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 174 | self.admin_client.update_router_with_snat_gw_info( |
| 175 | router['id'], |
| 176 | external_gateway_info={ |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 177 | 'network_id': CONF.network.public_network_id, |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 178 | 'enable_snat': False}) |
| 179 | self._verify_router_gateway( |
| 180 | router['id'], |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 181 | {'network_id': CONF.network.public_network_id, |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 182 | 'enable_snat': False}) |
| 183 | self._verify_gateway_port(router['id']) |
| 184 | |
| armando-migliaccio | 89e5b96 | 2013-12-16 06:12:16 -0800 | [diff] [blame] | 185 | @test.attr(type='smoke') |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 186 | def test_update_router_unset_gateway(self): |
| 187 | router = self.create_router( |
| Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 188 | data_utils.rand_name('router-'), |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 189 | external_network_id=CONF.network.public_network_id) |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 190 | self.client.update_router(router['id'], external_gateway_info={}) |
| 191 | self._verify_router_gateway(router['id']) |
| 192 | # No gateway port expected |
| 193 | resp, list_body = self.admin_client.list_ports( |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 194 | network_id=CONF.network.public_network_id, |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 195 | device_id=router['id']) |
| 196 | self.assertFalse(list_body['ports']) |
| 197 | |
| ronak | 79f1793 | 2014-01-27 17:56:21 -0800 | [diff] [blame] | 198 | @test.requires_ext(extension='ext-gw-mode', service='network') |
| armando-migliaccio | 89e5b96 | 2013-12-16 06:12:16 -0800 | [diff] [blame] | 199 | @test.attr(type='smoke') |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 200 | def test_update_router_reset_gateway_without_snat(self): |
| 201 | router = self.create_router( |
| Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 202 | data_utils.rand_name('router-'), |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 203 | external_network_id=CONF.network.public_network_id) |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 204 | self.admin_client.update_router_with_snat_gw_info( |
| 205 | router['id'], |
| 206 | external_gateway_info={ |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 207 | 'network_id': CONF.network.public_network_id, |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 208 | 'enable_snat': False}) |
| 209 | self._verify_router_gateway( |
| 210 | router['id'], |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 211 | {'network_id': CONF.network.public_network_id, |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 212 | 'enable_snat': False}) |
| 213 | self._verify_gateway_port(router['id']) |
| ChenZheng | 04ac694 | 2013-12-09 16:42:52 +0800 | [diff] [blame] | 214 | |
| armando-migliaccio | 89e5b96 | 2013-12-16 06:12:16 -0800 | [diff] [blame] | 215 | @test.requires_ext(extension='extraroute', service='network') |
| 216 | @test.attr(type='smoke') |
| ChenZheng | 04ac694 | 2013-12-09 16:42:52 +0800 | [diff] [blame] | 217 | def test_update_extra_route(self): |
| 218 | self.network = self.create_network() |
| 219 | self.name = self.network['name'] |
| 220 | self.subnet = self.create_subnet(self.network) |
| 221 | # Add router interface with subnet id |
| 222 | self.router = self.create_router(data_utils.rand_name('router-'), True) |
| 223 | self.create_router_interface(self.router['id'], self.subnet['id']) |
| 224 | self.addCleanup( |
| 225 | self._delete_extra_routes, |
| 226 | self.router['id']) |
| 227 | # Update router extra route |
| 228 | cidr = netaddr.IPNetwork(self.subnet['cidr']) |
| 229 | resp, extra_route = self.client.update_extra_routes( |
| 230 | self.router['id'], str(cidr[0]), str(self.subnet['cidr'])) |
| 231 | |
| 232 | def _delete_extra_routes(self, router_id): |
| 233 | resp, _ = self.client.delete_extra_routes(router_id) |