| 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 |
| zhufl | 6b7040a | 2017-01-18 16:38:34 +0800 | [diff] [blame] | 17 | import testtools |
| ChenZheng | 04ac694 | 2013-12-09 16:42:52 +0800 | [diff] [blame] | 18 | |
| Ken'ichi Ohmichi | 3fd5f0e | 2017-05-04 18:40:36 -0700 | [diff] [blame] | 19 | from tempest.api.network import base |
| Andrea Frittoli | cd36841 | 2017-08-14 21:37:56 +0100 | [diff] [blame] | 20 | from tempest.common import utils |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 21 | from tempest import config |
| Ken'ichi Ohmichi | f50e4df | 2017-03-10 10:52:53 -0800 | [diff] [blame] | 22 | from tempest.lib.common.utils import data_utils |
| Ken'ichi Ohmichi | 53b9a63 | 2017-01-27 18:04:39 -0800 | [diff] [blame] | 23 | from tempest.lib import decorators |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 24 | |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 25 | CONF = config.CONF |
| 26 | |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 27 | |
| Ken'ichi Ohmichi | 3204a0e | 2017-05-04 18:44:28 -0700 | [diff] [blame] | 28 | class RoutersTest(base.BaseNetworkTest): |
| Ken'ichi Ohmichi | 3fd5f0e | 2017-05-04 18:40:36 -0700 | [diff] [blame] | 29 | |
| 30 | def _cleanup_router(self, router): |
| 31 | self.delete_router(router) |
| 32 | self.routers.remove(router) |
| 33 | |
| 34 | def _create_router(self, name=None, admin_state_up=False, |
| 35 | external_network_id=None, enable_snat=None): |
| 36 | # associate a cleanup with created routers to avoid quota limits |
| 37 | router = self.create_router(name, admin_state_up, |
| 38 | external_network_id, enable_snat) |
| 39 | self.addCleanup(self._cleanup_router, router) |
| 40 | return router |
| 41 | |
| 42 | def _add_router_interface_with_subnet_id(self, router_id, subnet_id): |
| 43 | interface = self.routers_client.add_router_interface( |
| 44 | router_id, subnet_id=subnet_id) |
| 45 | self.addCleanup(self._remove_router_interface_with_subnet_id, |
| 46 | router_id, subnet_id) |
| 47 | self.assertEqual(subnet_id, interface['subnet_id']) |
| 48 | return interface |
| 49 | |
| 50 | def _remove_router_interface_with_subnet_id(self, router_id, subnet_id): |
| 51 | body = self.routers_client.remove_router_interface(router_id, |
| 52 | subnet_id=subnet_id) |
| 53 | self.assertEqual(subnet_id, body['subnet_id']) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 54 | |
| 55 | @classmethod |
| Rohan Kanade | a565e45 | 2015-01-27 14:00:13 +0530 | [diff] [blame] | 56 | def skip_checks(cls): |
| 57 | super(RoutersTest, cls).skip_checks() |
| Andrea Frittoli | cd36841 | 2017-08-14 21:37:56 +0100 | [diff] [blame] | 58 | if not utils.is_extension_enabled('router', 'network'): |
| Yoshihiro Kaneko | 0567026 | 2014-01-18 19:22:44 +0900 | [diff] [blame] | 59 | msg = "router extension not enabled." |
| 60 | raise cls.skipException(msg) |
| Rohan Kanade | a565e45 | 2015-01-27 14:00:13 +0530 | [diff] [blame] | 61 | |
| Jordan Pittier | 3b46d27 | 2017-04-12 16:17:28 +0200 | [diff] [blame] | 62 | @decorators.attr(type='smoke') |
| Ken'ichi Ohmichi | 53b9a63 | 2017-01-27 18:04:39 -0800 | [diff] [blame] | 63 | @decorators.idempotent_id('f64403e2-8483-4b34-8ccd-b09a87bcc68c') |
| zhufl | 6b7040a | 2017-01-18 16:38:34 +0800 | [diff] [blame] | 64 | @testtools.skipUnless(CONF.network.public_network_id, |
| 65 | 'The public_network_id option must be specified.') |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 66 | def test_create_show_list_update_delete_router(self): |
| 67 | # Create a router |
| zhufl | 3484f99 | 2017-10-10 16:18:29 +0800 | [diff] [blame] | 68 | name = data_utils.rand_name(self.__class__.__name__ + '-router') |
| Tianbiao Qi | 8315891 | 2016-11-03 10:37:30 +0800 | [diff] [blame] | 69 | router = self._create_router( |
| zhufl | 3484f99 | 2017-10-10 16:18:29 +0800 | [diff] [blame] | 70 | name=name, |
| Tianbiao Qi | 8315891 | 2016-11-03 10:37:30 +0800 | [diff] [blame] | 71 | admin_state_up=False, |
| 72 | external_network_id=CONF.network.public_network_id) |
| zhufl | 3484f99 | 2017-10-10 16:18:29 +0800 | [diff] [blame] | 73 | self.assertEqual(router['name'], name) |
| Tianbiao Qi | 8315891 | 2016-11-03 10:37:30 +0800 | [diff] [blame] | 74 | self.assertEqual(router['admin_state_up'], False) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 75 | self.assertEqual( |
| Tianbiao Qi | 8315891 | 2016-11-03 10:37:30 +0800 | [diff] [blame] | 76 | router['external_gateway_info']['network_id'], |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 77 | CONF.network.public_network_id) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 78 | # Show details of the created router |
| Tianbiao Qi | 8315891 | 2016-11-03 10:37:30 +0800 | [diff] [blame] | 79 | router_show = self.routers_client.show_router( |
| 80 | router['id'])['router'] |
| 81 | self.assertEqual(router_show['name'], router['name']) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 82 | self.assertEqual( |
| Tianbiao Qi | 8315891 | 2016-11-03 10:37:30 +0800 | [diff] [blame] | 83 | router_show['external_gateway_info']['network_id'], |
| Matthew Treinish | 03b48df | 2014-01-29 16:59:49 +0000 | [diff] [blame] | 84 | CONF.network.public_network_id) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 85 | # List routers and verify if created router is there in response |
| Tianbiao Qi | 8315891 | 2016-11-03 10:37:30 +0800 | [diff] [blame] | 86 | routers = self.routers_client.list_routers()['routers'] |
| 87 | self.assertIn(router['id'], map(lambda x: x['id'], routers)) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 88 | # Update the name of router and verify if it is updated |
| Tianbiao Qi | 8315891 | 2016-11-03 10:37:30 +0800 | [diff] [blame] | 89 | updated_name = 'updated' + router['name'] |
| 90 | router_update = self.routers_client.update_router( |
| 91 | router['id'], name=updated_name)['router'] |
| 92 | self.assertEqual(router_update['name'], updated_name) |
| 93 | router_show = self.routers_client.show_router( |
| 94 | router['id'])['router'] |
| 95 | self.assertEqual(router_show['name'], updated_name) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 96 | |
| Jordan Pittier | 3b46d27 | 2017-04-12 16:17:28 +0200 | [diff] [blame] | 97 | @decorators.attr(type='smoke') |
| Ken'ichi Ohmichi | 53b9a63 | 2017-01-27 18:04:39 -0800 | [diff] [blame] | 98 | @decorators.idempotent_id('b42e6e39-2e37-49cc-a6f4-8467e940900a') |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 99 | def test_add_remove_router_interface_with_subnet_id(self): |
| 100 | network = self.create_network() |
| 101 | subnet = self.create_subnet(network) |
| zhufl | d2c40ca | 2016-10-18 17:03:07 +0800 | [diff] [blame] | 102 | router = self._create_router() |
| Chang Bo Guo | f099f80 | 2013-09-13 19:01:46 -0700 | [diff] [blame] | 103 | # Add router interface with subnet id |
| Ken'ichi Ohmichi | e35f472 | 2015-12-22 04:57:11 +0000 | [diff] [blame] | 104 | interface = self.routers_client.add_router_interface( |
| 105 | router['id'], subnet_id=subnet['id']) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 106 | self.addCleanup(self._remove_router_interface_with_subnet_id, |
| Salvatore Orlando | a85e8fe | 2013-09-20 03:48:02 -0700 | [diff] [blame] | 107 | router['id'], subnet['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 |
| John Warren | 49c0fe5 | 2015-10-22 12:35:54 -0400 | [diff] [blame] | 111 | show_port_body = self.ports_client.show_port( |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 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']) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 115 | |
| Jordan Pittier | 3b46d27 | 2017-04-12 16:17:28 +0200 | [diff] [blame] | 116 | @decorators.attr(type='smoke') |
| Ken'ichi Ohmichi | 53b9a63 | 2017-01-27 18:04:39 -0800 | [diff] [blame] | 117 | @decorators.idempotent_id('2b7d2f37-6748-4d78-92e5-1d590234f0d5') |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 118 | def test_add_remove_router_interface_with_port_id(self): |
| 119 | network = self.create_network() |
| 120 | self.create_subnet(network) |
| zhufl | d2c40ca | 2016-10-18 17:03:07 +0800 | [diff] [blame] | 121 | router = self._create_router() |
| John Warren | 49c0fe5 | 2015-10-22 12:35:54 -0400 | [diff] [blame] | 122 | port_body = self.ports_client.create_port( |
| Eugene Nikanorov | e9d255a | 2013-12-18 16:31:42 +0400 | [diff] [blame] | 123 | network_id=network['id']) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 124 | # add router interface to port created above |
| Ken'ichi Ohmichi | e35f472 | 2015-12-22 04:57:11 +0000 | [diff] [blame] | 125 | interface = self.routers_client.add_router_interface( |
| piyush110786 | 94aca95 | 2015-12-17 12:54:44 +0530 | [diff] [blame] | 126 | router['id'], |
| 127 | port_id=port_body['port']['id']) |
| Ken'ichi Ohmichi | ea2a31a | 2017-05-04 14:08:07 -0700 | [diff] [blame] | 128 | self.addCleanup(self.routers_client.remove_router_interface, |
| 129 | router['id'], port_id=port_body['port']['id']) |
| Attila Fazekas | 4b8b008 | 2013-10-25 14:24:32 +0200 | [diff] [blame] | 130 | self.assertIn('subnet_id', interface.keys()) |
| 131 | self.assertIn('port_id', interface.keys()) |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 132 | # Verify router id is equal to device id in port details |
| John Warren | 49c0fe5 | 2015-10-22 12:35:54 -0400 | [diff] [blame] | 133 | show_port_body = self.ports_client.show_port( |
| nayna-patel | 197c012 | 2013-07-11 10:18:00 +0000 | [diff] [blame] | 134 | interface['port_id']) |
| 135 | self.assertEqual(show_port_body['port']['device_id'], |
| Salvatore Orlando | a85e8fe | 2013-09-20 03:48:02 -0700 | [diff] [blame] | 136 | router['id']) |
| Salvatore Orlando | ce22b49 | 2013-09-20 04:04:11 -0700 | [diff] [blame] | 137 | |
| lianghao | 8c8074c | 2017-03-09 22:16:52 +0800 | [diff] [blame] | 138 | @decorators.idempotent_id('cbe42f84-04c2-11e7-8adb-fa163e4fa634') |
| Andrea Frittoli | cd36841 | 2017-08-14 21:37:56 +0100 | [diff] [blame] | 139 | @utils.requires_ext(extension='ext-gw-mode', service='network') |
| lianghao | 8c8074c | 2017-03-09 22:16:52 +0800 | [diff] [blame] | 140 | @testtools.skipUnless(CONF.network.public_network_id, |
| 141 | 'The public_network_id option must be specified.') |
| Jordan Pittier | 7252ce0 | 2017-03-26 20:48:14 +0200 | [diff] [blame] | 142 | @decorators.skip_because(bug='1676207') |
| lianghao | 8c8074c | 2017-03-09 22:16:52 +0800 | [diff] [blame] | 143 | def test_create_router_set_gateway_with_fixed_ip(self): |
| 144 | # Don't know public_network_address, so at first create address |
| 145 | # from public_network and delete |
| 146 | port = self.admin_ports_client.create_port( |
| 147 | network_id=CONF.network.public_network_id)['port'] |
| 148 | self.admin_ports_client.delete_port(port_id=port['id']) |
| 149 | |
| 150 | fixed_ip = { |
| 151 | 'subnet_id': port['fixed_ips'][0]['subnet_id'], |
| 152 | 'ip_address': port['fixed_ips'][0]['ip_address'] |
| 153 | } |
| 154 | external_gateway_info = { |
| 155 | 'network_id': CONF.network.public_network_id, |
| 156 | 'external_fixed_ips': [fixed_ip] |
| 157 | } |
| 158 | |
| 159 | # Create a router and set gateway to fixed_ip |
| 160 | router = self.admin_routers_client.create_router( |
| 161 | external_gateway_info=external_gateway_info)['router'] |
| 162 | self.addCleanup(self.admin_routers_client.delete_router, |
| 163 | router_id=router['id']) |
| 164 | # Examine router's gateway is equal to fixed_ip |
| 165 | self.assertEqual(router['external_gateway_info'][ |
| 166 | 'external_fixed_ips'][0]['ip_address'], |
| 167 | fixed_ip['ip_address']) |
| 168 | |
| Ken'ichi Ohmichi | 53b9a63 | 2017-01-27 18:04:39 -0800 | [diff] [blame] | 169 | @decorators.idempotent_id('c86ac3a8-50bd-4b00-a6b8-62af84a0765c') |
| Andrea Frittoli | cd36841 | 2017-08-14 21:37:56 +0100 | [diff] [blame] | 170 | @utils.requires_ext(extension='extraroute', service='network') |
| Abhishek | 6707b5e | 2016-01-14 15:49:11 +0530 | [diff] [blame] | 171 | def test_update_delete_extra_route(self): |
| Shuquan Huang | de79d26 | 2015-06-26 02:48:22 +0000 | [diff] [blame] | 172 | # Create different cidr for each subnet to avoid cidr duplicate |
| Sean Dague | ed6e586 | 2016-04-04 10:49:13 -0400 | [diff] [blame] | 173 | # The cidr starts from project_cidr |
| zhufl | b0dbe4b | 2017-09-11 16:45:17 +0800 | [diff] [blame] | 174 | next_cidr = self.cidr |
| Shuquan Huang | de79d26 | 2015-06-26 02:48:22 +0000 | [diff] [blame] | 175 | # Prepare to build several routes |
| 176 | test_routes = [] |
| David Kranz | 4d06ebb | 2015-07-31 15:18:08 -0400 | [diff] [blame] | 177 | routes_num = 4 |
| Shuquan Huang | de79d26 | 2015-06-26 02:48:22 +0000 | [diff] [blame] | 178 | # Create a router |
| zhufl | d2c40ca | 2016-10-18 17:03:07 +0800 | [diff] [blame] | 179 | router = self._create_router(admin_state_up=True) |
| ChenZheng | 04ac694 | 2013-12-09 16:42:52 +0800 | [diff] [blame] | 180 | self.addCleanup( |
| 181 | self._delete_extra_routes, |
| Brandon Palm | f35aa8b | 2015-08-07 14:56:59 -0500 | [diff] [blame] | 182 | router['id']) |
| Sylvain Afchain | 85c1e54 | 2014-01-11 08:23:16 +0100 | [diff] [blame] | 183 | # Update router extra route, second ip of the range is |
| 184 | # used as next hop |
| Shuquan Huang | de79d26 | 2015-06-26 02:48:22 +0000 | [diff] [blame] | 185 | for i in range(routes_num): |
| 186 | network = self.create_network() |
| 187 | subnet = self.create_subnet(network, cidr=next_cidr) |
| 188 | next_cidr = next_cidr.next() |
| 189 | |
| 190 | # Add router interface with subnet id |
| Brandon Palm | f35aa8b | 2015-08-07 14:56:59 -0500 | [diff] [blame] | 191 | self.create_router_interface(router['id'], subnet['id']) |
| Shuquan Huang | de79d26 | 2015-06-26 02:48:22 +0000 | [diff] [blame] | 192 | |
| 193 | cidr = netaddr.IPNetwork(subnet['cidr']) |
| 194 | next_hop = str(cidr[2]) |
| 195 | destination = str(subnet['cidr']) |
| 196 | test_routes.append( |
| 197 | {'nexthop': next_hop, 'destination': destination} |
| 198 | ) |
| 199 | |
| 200 | test_routes.sort(key=lambda x: x['destination']) |
| Ken'ichi Ohmichi | 50fc784 | 2016-05-30 11:21:08 -0700 | [diff] [blame] | 201 | extra_route = self.routers_client.update_router( |
| Ken'ichi Ohmichi | e35f472 | 2015-12-22 04:57:11 +0000 | [diff] [blame] | 202 | router['id'], routes=test_routes) |
| 203 | show_body = self.routers_client.show_router(router['id']) |
| Shuquan Huang | de79d26 | 2015-06-26 02:48:22 +0000 | [diff] [blame] | 204 | # Assert the number of routes |
| 205 | self.assertEqual(routes_num, len(extra_route['router']['routes'])) |
| 206 | self.assertEqual(routes_num, len(show_body['router']['routes'])) |
| 207 | |
| 208 | routes = extra_route['router']['routes'] |
| 209 | routes.sort(key=lambda x: x['destination']) |
| 210 | # Assert the nexthops & destination |
| 211 | for i in range(routes_num): |
| 212 | self.assertEqual(test_routes[i]['destination'], |
| 213 | routes[i]['destination']) |
| 214 | self.assertEqual(test_routes[i]['nexthop'], routes[i]['nexthop']) |
| 215 | |
| 216 | routes = show_body['router']['routes'] |
| 217 | routes.sort(key=lambda x: x['destination']) |
| 218 | for i in range(routes_num): |
| 219 | self.assertEqual(test_routes[i]['destination'], |
| 220 | routes[i]['destination']) |
| 221 | self.assertEqual(test_routes[i]['nexthop'], routes[i]['nexthop']) |
| ChenZheng | 04ac694 | 2013-12-09 16:42:52 +0800 | [diff] [blame] | 222 | |
| Ken'ichi Ohmichi | 1f38761 | 2016-05-30 11:28:06 -0700 | [diff] [blame] | 223 | self._delete_extra_routes(router['id']) |
| Ken'ichi Ohmichi | e35f472 | 2015-12-22 04:57:11 +0000 | [diff] [blame] | 224 | show_body_after_deletion = self.routers_client.show_router( |
| 225 | router['id']) |
| Abhishek | 6707b5e | 2016-01-14 15:49:11 +0530 | [diff] [blame] | 226 | self.assertEmpty(show_body_after_deletion['router']['routes']) |
| 227 | |
| ChenZheng | 04ac694 | 2013-12-09 16:42:52 +0800 | [diff] [blame] | 228 | def _delete_extra_routes(self, router_id): |
| Ken'ichi Ohmichi | 1f38761 | 2016-05-30 11:28:06 -0700 | [diff] [blame] | 229 | self.routers_client.update_router(router_id, routes=None) |
| liudong | fbdfc7e | 2014-02-15 10:52:17 +0800 | [diff] [blame] | 230 | |
| Ken'ichi Ohmichi | 53b9a63 | 2017-01-27 18:04:39 -0800 | [diff] [blame] | 231 | @decorators.idempotent_id('a8902683-c788-4246-95c7-ad9c6d63a4d9') |
| liudong | fbdfc7e | 2014-02-15 10:52:17 +0800 | [diff] [blame] | 232 | def test_update_router_admin_state(self): |
| zhufl | d2c40ca | 2016-10-18 17:03:07 +0800 | [diff] [blame] | 233 | router = self._create_router() |
| Brandon Palm | f35aa8b | 2015-08-07 14:56:59 -0500 | [diff] [blame] | 234 | self.assertFalse(router['admin_state_up']) |
| liudong | fbdfc7e | 2014-02-15 10:52:17 +0800 | [diff] [blame] | 235 | # Update router admin state |
| Ken'ichi Ohmichi | e35f472 | 2015-12-22 04:57:11 +0000 | [diff] [blame] | 236 | update_body = self.routers_client.update_router(router['id'], |
| 237 | admin_state_up=True) |
| liudong | fbdfc7e | 2014-02-15 10:52:17 +0800 | [diff] [blame] | 238 | self.assertTrue(update_body['router']['admin_state_up']) |
| Ken'ichi Ohmichi | e35f472 | 2015-12-22 04:57:11 +0000 | [diff] [blame] | 239 | show_body = self.routers_client.show_router(router['id']) |
| liudong | fbdfc7e | 2014-02-15 10:52:17 +0800 | [diff] [blame] | 240 | self.assertTrue(show_body['router']['admin_state_up']) |
| 241 | |
| Jordan Pittier | 3b46d27 | 2017-04-12 16:17:28 +0200 | [diff] [blame] | 242 | @decorators.attr(type='smoke') |
| Ken'ichi Ohmichi | 53b9a63 | 2017-01-27 18:04:39 -0800 | [diff] [blame] | 243 | @decorators.idempotent_id('802c73c9-c937-4cef-824b-2191e24a6aab') |
| liudong | fbdfc7e | 2014-02-15 10:52:17 +0800 | [diff] [blame] | 244 | def test_add_multiple_router_interfaces(self): |
| armando-migliaccio | ee90a4d | 2014-05-06 11:54:07 -0700 | [diff] [blame] | 245 | network01 = self.create_network( |
| 246 | network_name=data_utils.rand_name('router-network01-')) |
| 247 | network02 = self.create_network( |
| 248 | network_name=data_utils.rand_name('router-network02-')) |
| 249 | subnet01 = self.create_subnet(network01) |
| zhufl | b0dbe4b | 2017-09-11 16:45:17 +0800 | [diff] [blame] | 250 | sub02_cidr = self.cidr.next() |
| armando-migliaccio | ee90a4d | 2014-05-06 11:54:07 -0700 | [diff] [blame] | 251 | subnet02 = self.create_subnet(network02, cidr=sub02_cidr) |
| zhufl | d2c40ca | 2016-10-18 17:03:07 +0800 | [diff] [blame] | 252 | router = self._create_router() |
| liudong | fbdfc7e | 2014-02-15 10:52:17 +0800 | [diff] [blame] | 253 | interface01 = self._add_router_interface_with_subnet_id(router['id'], |
| 254 | subnet01['id']) |
| 255 | self._verify_router_interface(router['id'], subnet01['id'], |
| 256 | interface01['port_id']) |
| 257 | interface02 = self._add_router_interface_with_subnet_id(router['id'], |
| 258 | subnet02['id']) |
| 259 | self._verify_router_interface(router['id'], subnet02['id'], |
| 260 | interface02['port_id']) |
| 261 | |
| Ken'ichi Ohmichi | 53b9a63 | 2017-01-27 18:04:39 -0800 | [diff] [blame] | 262 | @decorators.idempotent_id('96522edf-b4b5-45d9-8443-fa11c26e6eff') |
| Hardik Italia | e1a1707 | 2015-07-15 17:58:24 -0700 | [diff] [blame] | 263 | def test_router_interface_port_update_with_fixed_ip(self): |
| 264 | network = self.create_network() |
| 265 | subnet = self.create_subnet(network) |
| zhufl | d2c40ca | 2016-10-18 17:03:07 +0800 | [diff] [blame] | 266 | router = self._create_router() |
| Hardik Italia | e1a1707 | 2015-07-15 17:58:24 -0700 | [diff] [blame] | 267 | fixed_ip = [{'subnet_id': subnet['id']}] |
| 268 | interface = self._add_router_interface_with_subnet_id(router['id'], |
| 269 | subnet['id']) |
| 270 | self.assertIn('port_id', interface) |
| 271 | self.assertIn('subnet_id', interface) |
| 272 | port = self.ports_client.show_port(interface['port_id']) |
| 273 | self.assertEqual(port['port']['id'], interface['port_id']) |
| 274 | router_port = self.ports_client.update_port(port['port']['id'], |
| 275 | fixed_ips=fixed_ip) |
| 276 | self.assertEqual(subnet['id'], |
| 277 | router_port['port']['fixed_ips'][0]['subnet_id']) |
| 278 | |
| liudong | fbdfc7e | 2014-02-15 10:52:17 +0800 | [diff] [blame] | 279 | def _verify_router_interface(self, router_id, subnet_id, port_id): |
| John Warren | 49c0fe5 | 2015-10-22 12:35:54 -0400 | [diff] [blame] | 280 | show_port_body = self.ports_client.show_port(port_id) |
| liudong | fbdfc7e | 2014-02-15 10:52:17 +0800 | [diff] [blame] | 281 | interface_port = show_port_body['port'] |
| 282 | self.assertEqual(router_id, interface_port['device_id']) |
| 283 | self.assertEqual(subnet_id, |
| 284 | interface_port['fixed_ips'][0]['subnet_id']) |
| Sergey Shnaidman | 97e6a0f | 2014-11-12 20:00:53 +0300 | [diff] [blame] | 285 | |
| 286 | |
| 287 | class RoutersIpV6Test(RoutersTest): |
| 288 | _ip_version = 6 |