openstack/neutron-tempest-plugin
Code Issues Proposed changes

Execute router commands with admin client (2)

Browse Source 
The API calls that change the router external gateway info "enable_snat"
flag are, by default, allowed only for admin users. That fix is proposed
to the Neutron API definitions in [1].

This patch amends the missing bits of [2].

[1]https://review.opendev.org/c/openstack/neutron-lib/+/941631
[2]https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/941689

Related-Bug: #2098109
Change-Id: Ie496dec0e2396a232c4e4a891672e70f0d90857f
This commit is contained in:
Rodolfo Alonso Hernandez 2025-03-01 12:54:27 +00:00
parent eb7f7b0848
commit 658882f795
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
neutron_tempest_plugin/api/test_routers.py
View File

@ -377,7 +377,7 @@ class ExternalGWMultihomingRoutersTest(base_routers.BaseRouterTest):
router = self._create_router(data_utils.rand_name('router'))
self.assertEqual(len(router['external_gateways']), 0)
res = self.client.router_add_external_gateways(
res = self.admin_client.router_add_external_gateways(
router['id'],
[{'network_id': CONF.network.public_network_id,
'enable_snat': False}])
@ -391,7 +391,7 @@ class ExternalGWMultihomingRoutersTest(base_routers.BaseRouterTest):
router = self._create_router(data_utils.rand_name('router'))
self.assertEqual(len(router['external_gateways']), 0)
res = self.client.router_add_external_gateways(
res = self.admin_client.router_add_external_gateways(
router['id'],
[
{'network_id': CONF.network.public_network_id,
@ -452,7 +452,7 @@ class ExternalGWMultihomingRoutersTest(base_routers.BaseRouterTest):
router = self._create_router(data_utils.rand_name('router'))
self.assertEqual(len(router['external_gateways']), 0)
res = self.client.router_add_external_gateways(
res = self.admin_client.router_add_external_gateways(
router['id'],
[
{'network_id': CONF.network.public_network_id,
@ -483,7 +483,7 @@ class ExternalGWMultihomingRoutersTest(base_routers.BaseRouterTest):
router = self._create_router(data_utils.rand_name('router'))
self.assertEqual(len(router['external_gateways']), 0)
res = self.client.router_add_external_gateways(
res = self.admin_client.router_add_external_gateways(
router['id'],
[
{'network_id': CONF.network.public_network_id,
@ -506,7 +506,7 @@ class ExternalGWMultihomingRoutersTest(base_routers.BaseRouterTest):
remove_gateways[0])
external_gateways[1] = remove_gateways[0]
res_update_gws = self.client.router_update_external_gateways(
res_update_gws = self.admin_client.router_update_external_gateways(
router['id'],
external_gateways)