| 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 |
| jun xie | 82fbc58 | 2014-02-19 23:07:59 +0800 | [diff] [blame] | 19 | from tempest import clients |
| Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 20 | from tempest.common.utils import data_utils |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 21 | from tempest import config |
| armando-migliaccio | 89e5b96 | 2013-12-16 06:12:16 -0800 | [diff] [blame] | 22 | from tempest import test |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 23 | |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 24 | CONF = config.CONF |
| 25 | |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 26 | |
| rossella | e02431e | 2013-11-15 17:58:29 +0100 | [diff] [blame] | 27 | class RoutersTest(base.BaseRouterTest): |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 28 | |
| 29 | @classmethod |
| Andrea Frittoli | da4a245 | 2014-09-15 13:12:08 +0100 | [diff] [blame] | 30 | def resource_setup(cls): |
| 31 | super(RoutersTest, cls).resource_setup() |
| Yoshihiro Kaneko | 0567026 | 2014-01-18 19:22:44 +0900 | [diff] [blame] | 32 | if not test.is_extension_enabled('router', 'network'): |
| 33 | msg = "router extension not enabled." |
| 34 | raise cls.skipException(msg) |
| jun xie | 82fbc58 | 2014-02-19 23:07:59 +0800 | [diff] [blame] | 35 | admin_manager = clients.AdminManager() |
| 36 | cls.identity_admin_client = admin_manager.identity_client |
| Sergey Shnaidman | 97e6a0f | 2014-11-12 20:00:53 +0300 | [diff] [blame] | 37 | cls.tenant_cidr = (CONF.network.tenant_network_cidr |
| 38 | if cls._ip_version == 4 else |
| 39 | CONF.network.tenant_network_v6_cidr) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 40 | |
| Adam Gandelman | 3e9d12b | 2014-04-02 17:04:19 -0700 | [diff] [blame] | 41 | def _cleanup_router(self, router): |
| 42 | self.delete_router(router) |
| 43 | self.routers.remove(router) |
| 44 | |
| 45 | def _create_router(self, name, admin_state_up=False, |
| 46 | external_network_id=None, enable_snat=None): |
| 47 | # associate a cleanup with created routers to avoid quota limits |
| 48 | router = self.create_router(name, admin_state_up, |
| 49 | external_network_id, enable_snat) |
| 50 | self.addCleanup(self._cleanup_router, router) |
| 51 | return router |
| 52 | |
| armando-migliaccio | 89e5b96 | 2013-12-16 06:12:16 -0800 | [diff] [blame] | 53 | @test.attr(type='smoke') |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 54 | def test_create_show_list_update_delete_router(self): |
| 55 | # Create a router |
| Salvatore Orlando | a85e8fe | 2013-09-20 03:48:02 -0700 | [diff] [blame] | 56 | # NOTE(salv-orlando): Do not invoke self.create_router |
| 57 | # as we need to check the response code |
| Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 58 | name = data_utils.rand_name('router-') |
| David Kranz | 34e8812 | 2014-12-11 15:24:05 -0500 | [diff] [blame] | 59 | create_body = self.client.create_router( |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 60 | name, external_gateway_info={ |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 61 | "network_id": CONF.network.public_network_id}, |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 62 | admin_state_up=False) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 63 | self.addCleanup(self._delete_router, create_body['router']['id']) |
| 64 | self.assertEqual(create_body['router']['name'], name) |
| 65 | self.assertEqual( |
| 66 | create_body['router']['external_gateway_info']['network_id'], |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 67 | CONF.network.public_network_id) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 68 | self.assertEqual(create_body['router']['admin_state_up'], False) |
| 69 | # Show details of the created router |
| David Kranz | 34e8812 | 2014-12-11 15:24:05 -0500 | [diff] [blame] | 70 | show_body = self.client.show_router(create_body['router']['id']) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 71 | self.assertEqual(show_body['router']['name'], name) |
| 72 | self.assertEqual( |
| 73 | show_body['router']['external_gateway_info']['network_id'], |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 74 | CONF.network.public_network_id) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 75 | self.assertEqual(show_body['router']['admin_state_up'], False) |
| 76 | # List routers and verify if created router is there in response |
| David Kranz | 34e8812 | 2014-12-11 15:24:05 -0500 | [diff] [blame] | 77 | list_body = self.client.list_routers() |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 78 | routers_list = list() |
| 79 | for router in list_body['routers']: |
| 80 | routers_list.append(router['id']) |
| 81 | self.assertIn(create_body['router']['id'], routers_list) |
| 82 | # Update the name of router and verify if it is updated |
| 83 | updated_name = 'updated ' + name |
| David Kranz | 34e8812 | 2014-12-11 15:24:05 -0500 | [diff] [blame] | 84 | update_body = self.client.update_router(create_body['router']['id'], |
| 85 | name=updated_name) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 86 | self.assertEqual(update_body['router']['name'], updated_name) |
| David Kranz | 34e8812 | 2014-12-11 15:24:05 -0500 | [diff] [blame] | 87 | show_body = self.client.show_router( |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 88 | create_body['router']['id']) |
| 89 | self.assertEqual(show_body['router']['name'], updated_name) |
| 90 | |
| armando-migliaccio | 89e5b96 | 2013-12-16 06:12:16 -0800 | [diff] [blame] | 91 | @test.attr(type='smoke') |
| jun xie | 82fbc58 | 2014-02-19 23:07:59 +0800 | [diff] [blame] | 92 | def test_create_router_setting_tenant_id(self): |
| 93 | # Test creating router from admin user setting tenant_id. |
| 94 | test_tenant = data_utils.rand_name('test_tenant_') |
| 95 | test_description = data_utils.rand_name('desc_') |
| David Kranz | b7afa92 | 2014-12-30 10:56:26 -0500 | [diff] [blame] | 96 | tenant = self.identity_admin_client.create_tenant( |
| Rohan Kanade | eeb2164 | 2014-08-14 12:00:26 +0200 | [diff] [blame] | 97 | name=test_tenant, description=test_description) |
| jun xie | 82fbc58 | 2014-02-19 23:07:59 +0800 | [diff] [blame] | 98 | tenant_id = tenant['id'] |
| 99 | self.addCleanup(self.identity_admin_client.delete_tenant, tenant_id) |
| 100 | |
| 101 | name = data_utils.rand_name('router-') |
| David Kranz | 34e8812 | 2014-12-11 15:24:05 -0500 | [diff] [blame] | 102 | create_body = self.admin_client.create_router(name, |
| 103 | tenant_id=tenant_id) |
| jun xie | 82fbc58 | 2014-02-19 23:07:59 +0800 | [diff] [blame] | 104 | self.addCleanup(self.admin_client.delete_router, |
| 105 | create_body['router']['id']) |
| 106 | self.assertEqual(tenant_id, create_body['router']['tenant_id']) |
| 107 | |
| varun yadav | faa3c8d | 2014-07-18 05:44:03 -0700 | [diff] [blame] | 108 | @test.requires_ext(extension='ext-gw-mode', service='network') |
| 109 | @test.attr(type='smoke') |
| 110 | def test_create_router_with_default_snat_value(self): |
| 111 | # Create a router with default snat rule |
| 112 | name = data_utils.rand_name('router') |
| Matthew Treinish | 1d14c54 | 2014-06-17 20:25:40 -0400 | [diff] [blame] | 113 | router = self._create_router( |
| 114 | name, external_network_id=CONF.network.public_network_id) |
| 115 | self._verify_router_gateway( |
| 116 | router['id'], {'network_id': CONF.network.public_network_id, |
| 117 | 'enable_snat': True}) |
| varun yadav | faa3c8d | 2014-07-18 05:44:03 -0700 | [diff] [blame] | 118 | |
| 119 | @test.requires_ext(extension='ext-gw-mode', service='network') |
| 120 | @test.attr(type='smoke') |
| 121 | def test_create_router_with_snat_explicit(self): |
| 122 | name = data_utils.rand_name('snat-router') |
| 123 | # Create a router enabling snat attributes |
| 124 | enable_snat_states = [False, True] |
| 125 | for enable_snat in enable_snat_states: |
| 126 | external_gateway_info = { |
| 127 | 'network_id': CONF.network.public_network_id, |
| 128 | 'enable_snat': enable_snat} |
| David Kranz | 34e8812 | 2014-12-11 15:24:05 -0500 | [diff] [blame] | 129 | create_body = self.admin_client.create_router( |
| varun yadav | faa3c8d | 2014-07-18 05:44:03 -0700 | [diff] [blame] | 130 | name, external_gateway_info=external_gateway_info) |
| varun yadav | faa3c8d | 2014-07-18 05:44:03 -0700 | [diff] [blame] | 131 | self.addCleanup(self.admin_client.delete_router, |
| Matthew Treinish | 1d14c54 | 2014-06-17 20:25:40 -0400 | [diff] [blame] | 132 | create_body['router']['id']) |
| varun yadav | faa3c8d | 2014-07-18 05:44:03 -0700 | [diff] [blame] | 133 | # Verify snat attributes after router creation |
| 134 | self._verify_router_gateway(create_body['router']['id'], |
| Matthew Treinish | 1d14c54 | 2014-06-17 20:25:40 -0400 | [diff] [blame] | 135 | exp_ext_gw_info=external_gateway_info) |
| varun yadav | faa3c8d | 2014-07-18 05:44:03 -0700 | [diff] [blame] | 136 | |
| jun xie | 82fbc58 | 2014-02-19 23:07:59 +0800 | [diff] [blame] | 137 | @test.attr(type='smoke') |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 138 | def test_add_remove_router_interface_with_subnet_id(self): |
| 139 | network = self.create_network() |
| 140 | subnet = self.create_subnet(network) |
| Adam Gandelman | 3e9d12b | 2014-04-02 17:04:19 -0700 | [diff] [blame] | 141 | router = self._create_router(data_utils.rand_name('router-')) |
| Chang Bo Guo | f099f80 | 2013-09-13 19:01:46 -0700 | [diff] [blame] | 142 | # Add router interface with subnet id |
| David Kranz | 34e8812 | 2014-12-11 15:24:05 -0500 | [diff] [blame] | 143 | interface = self.client.add_router_interface_with_subnet_id( |
| Salvatore Orlando | a85e8fe | 2013-09-20 03:48:02 -0700 | [diff] [blame] | 144 | router['id'], subnet['id']) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 145 | self.addCleanup(self._remove_router_interface_with_subnet_id, |
| Salvatore Orlando | a85e8fe | 2013-09-20 03:48:02 -0700 | [diff] [blame] | 146 | router['id'], subnet['id']) |
| Attila Fazekas | 4b8b008 | 2013-10-25 14:24:32 +0200 | [diff] [blame] | 147 | self.assertIn('subnet_id', interface.keys()) |
| 148 | self.assertIn('port_id', interface.keys()) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 149 | # Verify router id is equal to device id in port details |
| David Kranz | 34e8812 | 2014-12-11 15:24:05 -0500 | [diff] [blame] | 150 | show_port_body = self.client.show_port( |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 151 | interface['port_id']) |
| 152 | self.assertEqual(show_port_body['port']['device_id'], |
| Salvatore Orlando | a85e8fe | 2013-09-20 03:48:02 -0700 | [diff] [blame] | 153 | router['id']) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 154 | |
| armando-migliaccio | 89e5b96 | 2013-12-16 06:12:16 -0800 | [diff] [blame] | 155 | @test.attr(type='smoke') |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 156 | def test_add_remove_router_interface_with_port_id(self): |
| 157 | network = self.create_network() |
| 158 | self.create_subnet(network) |
| Adam Gandelman | 3e9d12b | 2014-04-02 17:04:19 -0700 | [diff] [blame] | 159 | router = self._create_router(data_utils.rand_name('router-')) |
| David Kranz | 34e8812 | 2014-12-11 15:24:05 -0500 | [diff] [blame] | 160 | port_body = self.client.create_port( |
| Eugene Nikanorov | e9d255a | 2013-12-18 16:31:42 +0400 | [diff] [blame] | 161 | network_id=network['id']) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 162 | # add router interface to port created above |
| David Kranz | 34e8812 | 2014-12-11 15:24:05 -0500 | [diff] [blame] | 163 | interface = self.client.add_router_interface_with_port_id( |
| Salvatore Orlando | a85e8fe | 2013-09-20 03:48:02 -0700 | [diff] [blame] | 164 | router['id'], port_body['port']['id']) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 165 | self.addCleanup(self._remove_router_interface_with_port_id, |
| Salvatore Orlando | a85e8fe | 2013-09-20 03:48:02 -0700 | [diff] [blame] | 166 | router['id'], port_body['port']['id']) |
| Attila Fazekas | 4b8b008 | 2013-10-25 14:24:32 +0200 | [diff] [blame] | 167 | self.assertIn('subnet_id', interface.keys()) |
| 168 | self.assertIn('port_id', interface.keys()) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 169 | # Verify router id is equal to device id in port details |
| David Kranz | 34e8812 | 2014-12-11 15:24:05 -0500 | [diff] [blame] | 170 | show_port_body = self.client.show_port( |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 171 | interface['port_id']) |
| 172 | self.assertEqual(show_port_body['port']['device_id'], |
| Salvatore Orlando | a85e8fe | 2013-09-20 03:48:02 -0700 | [diff] [blame] | 173 | router['id']) |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 174 | |
| 175 | def _verify_router_gateway(self, router_id, exp_ext_gw_info=None): |
| David Kranz | 34e8812 | 2014-12-11 15:24:05 -0500 | [diff] [blame] | 176 | show_body = self.admin_client.show_router(router_id) |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 177 | actual_ext_gw_info = show_body['router']['external_gateway_info'] |
| 178 | if exp_ext_gw_info is None: |
| 179 | self.assertIsNone(actual_ext_gw_info) |
| 180 | return |
| 181 | # Verify only keys passed in exp_ext_gw_info |
| 182 | for k, v in exp_ext_gw_info.iteritems(): |
| 183 | self.assertEqual(v, actual_ext_gw_info[k]) |
| 184 | |
| 185 | def _verify_gateway_port(self, router_id): |
| David Kranz | 34e8812 | 2014-12-11 15:24:05 -0500 | [diff] [blame] | 186 | list_body = self.admin_client.list_ports( |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 187 | network_id=CONF.network.public_network_id, |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 188 | device_id=router_id) |
| 189 | self.assertEqual(len(list_body['ports']), 1) |
| 190 | gw_port = list_body['ports'][0] |
| 191 | fixed_ips = gw_port['fixed_ips'] |
| Sean M. Collins | 08803f6 | 2014-12-09 16:41:01 -0700 | [diff] [blame] | 192 | self.assertGreaterEqual(len(fixed_ips), 1) |
| David Kranz | 34e8812 | 2014-12-11 15:24:05 -0500 | [diff] [blame] | 193 | public_net_body = self.admin_client.show_network( |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 194 | CONF.network.public_network_id) |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 195 | public_subnet_id = public_net_body['network']['subnets'][0] |
| Sean M. Collins | 2bc293e | 2014-12-10 10:37:57 -0700 | [diff] [blame] | 196 | self.assertIn(public_subnet_id, |
| 197 | map(lambda x: x['subnet_id'], fixed_ips)) |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 198 | |
| 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_set_gateway(self): |
| Adam Gandelman | 3e9d12b | 2014-04-02 17:04:19 -0700 | [diff] [blame] | 201 | router = self._create_router(data_utils.rand_name('router-')) |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 202 | self.client.update_router( |
| 203 | router['id'], |
| 204 | external_gateway_info={ |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 205 | 'network_id': CONF.network.public_network_id}) |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 206 | # Verify operation - router |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 207 | self._verify_router_gateway( |
| 208 | router['id'], |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 209 | {'network_id': CONF.network.public_network_id}) |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 210 | self._verify_gateway_port(router['id']) |
| 211 | |
| ronak | 79f1793 | 2014-01-27 17:56:21 -0800 | [diff] [blame] | 212 | @test.requires_ext(extension='ext-gw-mode', service='network') |
| armando-migliaccio | 89e5b96 | 2013-12-16 06:12:16 -0800 | [diff] [blame] | 213 | @test.attr(type='smoke') |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 214 | def test_update_router_set_gateway_with_snat_explicit(self): |
| Adam Gandelman | 3e9d12b | 2014-04-02 17:04:19 -0700 | [diff] [blame] | 215 | router = self._create_router(data_utils.rand_name('router-')) |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 216 | self.admin_client.update_router_with_snat_gw_info( |
| 217 | router['id'], |
| 218 | external_gateway_info={ |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 219 | 'network_id': CONF.network.public_network_id, |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 220 | 'enable_snat': True}) |
| 221 | self._verify_router_gateway( |
| 222 | router['id'], |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 223 | {'network_id': CONF.network.public_network_id, |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 224 | 'enable_snat': True}) |
| 225 | self._verify_gateway_port(router['id']) |
| 226 | |
| ronak | 79f1793 | 2014-01-27 17:56:21 -0800 | [diff] [blame] | 227 | @test.requires_ext(extension='ext-gw-mode', service='network') |
| armando-migliaccio | 89e5b96 | 2013-12-16 06:12:16 -0800 | [diff] [blame] | 228 | @test.attr(type='smoke') |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 229 | def test_update_router_set_gateway_without_snat(self): |
| Adam Gandelman | 3e9d12b | 2014-04-02 17:04:19 -0700 | [diff] [blame] | 230 | router = self._create_router(data_utils.rand_name('router-')) |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 231 | self.admin_client.update_router_with_snat_gw_info( |
| 232 | router['id'], |
| 233 | external_gateway_info={ |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 234 | 'network_id': CONF.network.public_network_id, |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 235 | 'enable_snat': False}) |
| 236 | self._verify_router_gateway( |
| 237 | router['id'], |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 238 | {'network_id': CONF.network.public_network_id, |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 239 | 'enable_snat': False}) |
| 240 | self._verify_gateway_port(router['id']) |
| 241 | |
| armando-migliaccio | 89e5b96 | 2013-12-16 06:12:16 -0800 | [diff] [blame] | 242 | @test.attr(type='smoke') |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 243 | def test_update_router_unset_gateway(self): |
| Adam Gandelman | 3e9d12b | 2014-04-02 17:04:19 -0700 | [diff] [blame] | 244 | router = self._create_router( |
| Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 245 | data_utils.rand_name('router-'), |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 246 | external_network_id=CONF.network.public_network_id) |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 247 | self.client.update_router(router['id'], external_gateway_info={}) |
| 248 | self._verify_router_gateway(router['id']) |
| 249 | # No gateway port expected |
| David Kranz | 34e8812 | 2014-12-11 15:24:05 -0500 | [diff] [blame] | 250 | list_body = self.admin_client.list_ports( |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 251 | network_id=CONF.network.public_network_id, |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 252 | device_id=router['id']) |
| 253 | self.assertFalse(list_body['ports']) |
| 254 | |
| ronak | 79f1793 | 2014-01-27 17:56:21 -0800 | [diff] [blame] | 255 | @test.requires_ext(extension='ext-gw-mode', service='network') |
| armando-migliaccio | 89e5b96 | 2013-12-16 06:12:16 -0800 | [diff] [blame] | 256 | @test.attr(type='smoke') |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 257 | def test_update_router_reset_gateway_without_snat(self): |
| Adam Gandelman | 3e9d12b | 2014-04-02 17:04:19 -0700 | [diff] [blame] | 258 | router = self._create_router( |
| Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 259 | data_utils.rand_name('router-'), |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 260 | external_network_id=CONF.network.public_network_id) |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 261 | self.admin_client.update_router_with_snat_gw_info( |
| 262 | router['id'], |
| 263 | external_gateway_info={ |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 264 | 'network_id': CONF.network.public_network_id, |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 265 | 'enable_snat': False}) |
| 266 | self._verify_router_gateway( |
| 267 | router['id'], |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 268 | {'network_id': CONF.network.public_network_id, |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 269 | 'enable_snat': False}) |
| 270 | self._verify_gateway_port(router['id']) |
| ChenZheng | 04ac694 | 2013-12-09 16:42:52 +0800 | [diff] [blame] | 271 | |
| armando-migliaccio | 89e5b96 | 2013-12-16 06:12:16 -0800 | [diff] [blame] | 272 | @test.requires_ext(extension='extraroute', service='network') |
| 273 | @test.attr(type='smoke') |
| ChenZheng | 04ac694 | 2013-12-09 16:42:52 +0800 | [diff] [blame] | 274 | def test_update_extra_route(self): |
| 275 | self.network = self.create_network() |
| 276 | self.name = self.network['name'] |
| 277 | self.subnet = self.create_subnet(self.network) |
| 278 | # Add router interface with subnet id |
| Adam Gandelman | 3e9d12b | 2014-04-02 17:04:19 -0700 | [diff] [blame] | 279 | self.router = self._create_router( |
| 280 | data_utils.rand_name('router-'), True) |
| ChenZheng | 04ac694 | 2013-12-09 16:42:52 +0800 | [diff] [blame] | 281 | self.create_router_interface(self.router['id'], self.subnet['id']) |
| 282 | self.addCleanup( |
| 283 | self._delete_extra_routes, |
| 284 | self.router['id']) |
| Sylvain Afchain | 85c1e54 | 2014-01-11 08:23:16 +0100 | [diff] [blame] | 285 | # Update router extra route, second ip of the range is |
| 286 | # used as next hop |
| ChenZheng | 04ac694 | 2013-12-09 16:42:52 +0800 | [diff] [blame] | 287 | cidr = netaddr.IPNetwork(self.subnet['cidr']) |
| Sylvain Afchain | 85c1e54 | 2014-01-11 08:23:16 +0100 | [diff] [blame] | 288 | next_hop = str(cidr[2]) |
| 289 | destination = str(self.subnet['cidr']) |
| David Kranz | 34e8812 | 2014-12-11 15:24:05 -0500 | [diff] [blame] | 290 | extra_route = self.client.update_extra_routes(self.router['id'], |
| 291 | next_hop, destination) |
| Sylvain Afchain | 85c1e54 | 2014-01-11 08:23:16 +0100 | [diff] [blame] | 292 | self.assertEqual(1, len(extra_route['router']['routes'])) |
| 293 | self.assertEqual(destination, |
| 294 | extra_route['router']['routes'][0]['destination']) |
| 295 | self.assertEqual(next_hop, |
| 296 | extra_route['router']['routes'][0]['nexthop']) |
| David Kranz | 34e8812 | 2014-12-11 15:24:05 -0500 | [diff] [blame] | 297 | show_body = self.client.show_router(self.router['id']) |
| Sylvain Afchain | 85c1e54 | 2014-01-11 08:23:16 +0100 | [diff] [blame] | 298 | self.assertEqual(destination, |
| 299 | show_body['router']['routes'][0]['destination']) |
| 300 | self.assertEqual(next_hop, |
| 301 | show_body['router']['routes'][0]['nexthop']) |
| ChenZheng | 04ac694 | 2013-12-09 16:42:52 +0800 | [diff] [blame] | 302 | |
| 303 | def _delete_extra_routes(self, router_id): |
| David Kranz | 34e8812 | 2014-12-11 15:24:05 -0500 | [diff] [blame] | 304 | self.client.delete_extra_routes(router_id) |
| liudong | fbdfc7e | 2014-02-15 10:52:17 +0800 | [diff] [blame] | 305 | |
| 306 | @test.attr(type='smoke') |
| 307 | def test_update_router_admin_state(self): |
| Adam Gandelman | 3e9d12b | 2014-04-02 17:04:19 -0700 | [diff] [blame] | 308 | self.router = self._create_router(data_utils.rand_name('router-')) |
| liudong | fbdfc7e | 2014-02-15 10:52:17 +0800 | [diff] [blame] | 309 | self.assertFalse(self.router['admin_state_up']) |
| 310 | # Update router admin state |
| David Kranz | 34e8812 | 2014-12-11 15:24:05 -0500 | [diff] [blame] | 311 | update_body = self.client.update_router(self.router['id'], |
| 312 | admin_state_up=True) |
| liudong | fbdfc7e | 2014-02-15 10:52:17 +0800 | [diff] [blame] | 313 | self.assertTrue(update_body['router']['admin_state_up']) |
| David Kranz | 34e8812 | 2014-12-11 15:24:05 -0500 | [diff] [blame] | 314 | show_body = self.client.show_router(self.router['id']) |
| liudong | fbdfc7e | 2014-02-15 10:52:17 +0800 | [diff] [blame] | 315 | self.assertTrue(show_body['router']['admin_state_up']) |
| 316 | |
| 317 | @test.attr(type='smoke') |
| 318 | def test_add_multiple_router_interfaces(self): |
| armando-migliaccio | ee90a4d | 2014-05-06 11:54:07 -0700 | [diff] [blame] | 319 | network01 = self.create_network( |
| 320 | network_name=data_utils.rand_name('router-network01-')) |
| 321 | network02 = self.create_network( |
| 322 | network_name=data_utils.rand_name('router-network02-')) |
| 323 | subnet01 = self.create_subnet(network01) |
| Sergey Shnaidman | 97e6a0f | 2014-11-12 20:00:53 +0300 | [diff] [blame] | 324 | sub02_cidr = netaddr.IPNetwork(self.tenant_cidr).next() |
| armando-migliaccio | ee90a4d | 2014-05-06 11:54:07 -0700 | [diff] [blame] | 325 | subnet02 = self.create_subnet(network02, cidr=sub02_cidr) |
| Adam Gandelman | 3e9d12b | 2014-04-02 17:04:19 -0700 | [diff] [blame] | 326 | router = self._create_router(data_utils.rand_name('router-')) |
| liudong | fbdfc7e | 2014-02-15 10:52:17 +0800 | [diff] [blame] | 327 | interface01 = self._add_router_interface_with_subnet_id(router['id'], |
| 328 | subnet01['id']) |
| 329 | self._verify_router_interface(router['id'], subnet01['id'], |
| 330 | interface01['port_id']) |
| 331 | interface02 = self._add_router_interface_with_subnet_id(router['id'], |
| 332 | subnet02['id']) |
| 333 | self._verify_router_interface(router['id'], subnet02['id'], |
| 334 | interface02['port_id']) |
| 335 | |
| liudong | fbdfc7e | 2014-02-15 10:52:17 +0800 | [diff] [blame] | 336 | def _verify_router_interface(self, router_id, subnet_id, port_id): |
| David Kranz | 34e8812 | 2014-12-11 15:24:05 -0500 | [diff] [blame] | 337 | show_port_body = self.client.show_port(port_id) |
| liudong | fbdfc7e | 2014-02-15 10:52:17 +0800 | [diff] [blame] | 338 | interface_port = show_port_body['port'] |
| 339 | self.assertEqual(router_id, interface_port['device_id']) |
| 340 | self.assertEqual(subnet_id, |
| 341 | interface_port['fixed_ips'][0]['subnet_id']) |
| Sergey Shnaidman | 97e6a0f | 2014-11-12 20:00:53 +0300 | [diff] [blame] | 342 | |
| 343 | |
| 344 | class RoutersIpV6Test(RoutersTest): |
| 345 | _ip_version = 6 |