Skip to content

Commit fab4fcc

Browse files
author
Dean Troyer
committed
Fix network create --project
SDK 0.9.13 needs to see project_id in the attributes, not tenant_id. Closes-bug: 1659878 Change-Id: Iff7abe8bd00cbe087c07579596c40af8b0a73302
1 parent ca76379 commit fab4fcc

2 files changed

Lines changed: 6 additions & 0 deletions

File tree

openstackclient/network/v2/network.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,9 @@ def _get_attrs(client_manager, parsed_args):
9393
parsed_args.project,
9494
parsed_args.project_domain,
9595
).id
96+
# TODO(dtroyer): Remove tenant_id when we clean up the SDK refactor
9697
attrs['tenant_id'] = project_id
98+
attrs['project_id'] = project_id
9799

98100
# "network set" command doesn't support setting availability zone hints.
99101
if 'availability_zone_hints' in parsed_args and \

openstackclient/tests/unit/network/v2/test_network.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -185,7 +185,9 @@ def test_create_all_options(self):
185185
'name': self._network.name,
186186
'shared': True,
187187
'description': self._network.description,
188+
# TODO(dtroyer): Remove tenant_id when we clean up the SDK refactor
188189
'tenant_id': self.project.id,
190+
'project_id': self.project.id,
189191
'is_default': True,
190192
'router:external': True,
191193
'provider:network_type': 'vlan',
@@ -319,7 +321,9 @@ def test_create_with_project_identityv2(self):
319321
self.network.create_network.assert_called_once_with(**{
320322
'admin_state_up': True,
321323
'name': self._network.name,
324+
# TODO(dtroyer): Remove tenant_id when we clean up the SDK refactor
322325
'tenant_id': self.project.id,
326+
'project_id': self.project.id,
323327
})
324328
self.assertEqual(self.columns, columns)
325329
self.assertEqual(self.data, data)

0 commit comments

Comments
 (0)