openstack/python-openstackclient
Code Issues Proposed changes

evacuate: Fix password parameter name for SDK

Browse Source 
The parameter is called admin_password on the SDK side.

Change-Id: I0cd86675a884e6c2cbd3a861b8e111f961f0f336
This commit is contained in:
Dr. Jens Harbott 2024-09-13 12:39:52 +02:00
parent ffa683ab4e
commit 8932282952
2 changed files with 6 additions and 6 deletions
openstackclient
compute/v2
server.py
tests/unit/compute/v2
test_server.py

2
openstackclient/compute/v2/server.py

@ -3840,7 +3840,7 @@ host."""
kwargs = { kwargs = {
'host': parsed_args.host, 'host': parsed_args.host,
'password': parsed_args.password, 'admin_password': parsed_args.password,
} }
if not sdk_utils.supports_microversion(compute_client, '2.14'): if not sdk_utils.supports_microversion(compute_client, '2.14'):

10
openstackclient/tests/unit/compute/v2/test_server.py

@ -6974,7 +6974,7 @@ class TestServerEvacuate(TestServer):
evac_args = { evac_args = {
'host': None, 'host': None,
'on_shared_storage': False, 'on_shared_storage': False,
'password': None, 'admin_password': None,
} }
self._test_evacuate(args, verify_args, evac_args) self._test_evacuate(args, verify_args, evac_args)
@ -6991,7 +6991,7 @@ class TestServerEvacuate(TestServer):
evac_args = { evac_args = {
'host': None, 'host': None,
'on_shared_storage': False, 'on_shared_storage': False,
'password': 'password', 'admin_password': 'password',
} }
self._test_evacuate(args, verify_args, evac_args) self._test_evacuate(args, verify_args, evac_args)
@ -7008,7 +7008,7 @@ class TestServerEvacuate(TestServer):
('server', self.server.id), ('server', self.server.id),
('host', 'target-host'), ('host', 'target-host'),
] ]
evac_args = {'host': host, 'password': None} evac_args = {'host': host, 'admin_password': None}
self._test_evacuate(args, verify_args, evac_args) self._test_evacuate(args, verify_args, evac_args)
@ -7041,7 +7041,7 @@ class TestServerEvacuate(TestServer):
evac_args = { evac_args = {
'host': None, 'host': None,
'on_shared_storage': True, 'on_shared_storage': True,
'password': None, 'admin_password': None,
} }
self._test_evacuate(args, verify_args, evac_args) self._test_evacuate(args, verify_args, evac_args)
@ -7072,7 +7072,7 @@ class TestServerEvacuate(TestServer):
evac_args = { evac_args = {
'host': None, 'host': None,
'on_shared_storage': False, 'on_shared_storage': False,
'password': None, 'admin_password': None,
} }
self._test_evacuate(args, verify_args, evac_args) self._test_evacuate(args, verify_args, evac_args)
mock_wait_for_status.assert_called_once_with( mock_wait_for_status.assert_called_once_with(