summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLingxian Kong <anlin.kong@gmail.com>2020-11-26 13:26:41 +1300
committerLingxian Kong <anlin.kong@gmail.com>2020-11-26 13:37:58 +1300
commit3647805c673d0124d72c01ad1eef2478825514fe (patch)
tree3b5c6800f14a0ea49700b6f8b35a4f3254b8c819
parentf28390ee5ddf83c0dfbf886b7e99c192fa7f4ecc (diff)
downloadpython-troveclient-3647805c673d0124d72c01ad1eef2478825514fe.tar.gz
Revert "Fix resize flavor for instance"
Story: 2008341 Task: 41244 This reverts commit f859692cc3f3f9ad72546d6b94f991a1558d78af. Change-Id: I690956927fb2558950362d1f223596db92e53d1e
-rw-r--r--troveclient/osc/v1/database_instances.py15
-rw-r--r--troveclient/tests/osc/v1/test_database_instances.py11
2 files changed, 15 insertions, 11 deletions
diff --git a/troveclient/osc/v1/database_instances.py b/troveclient/osc/v1/database_instances.py
index c99b776..fb751f6 100644
--- a/troveclient/osc/v1/database_instances.py
+++ b/troveclient/osc/v1/database_instances.py
@@ -509,21 +509,24 @@ class ResizeDatabaseInstanceFlavor(command.Command):
help=_('ID or name of the instance')
)
parser.add_argument(
- 'flavor_id',
- metavar='<flavor_id>',
+ 'flavor',
type=str,
- help=_('New flavor ID of the instance')
+ help=_('ID or name of the new flavor.')
)
return parser
def take_action(self, parsed_args):
instance_mgr = self.app.client_manager.database.instances
- instance_id = parsed_args.instance
+ flavor_mgr = self.app.client_manager.database.flavors
+ instance_id = parsed_args.instance
if not uuidutils.is_uuid_like(instance_id):
- instance_id = osc_utils.find_resource(instance_mgr, instance_id)
+ instance = osc_utils.find_resource(instance_mgr, instance_id)
+ instance_id = instance.id
+
+ flavor = osc_utils.find_resource(flavor_mgr, parsed_args.flavor)
- instance_mgr.resize_instance(instance_id, parsed_args.flavor_id)
+ instance_mgr.resize_instance(instance_id, flavor.id)
class UpgradeDatabaseInstance(command.Command):
diff --git a/troveclient/tests/osc/v1/test_database_instances.py b/troveclient/tests/osc/v1/test_database_instances.py
index af2134e..72202fe 100644
--- a/troveclient/tests/osc/v1/test_database_instances.py
+++ b/troveclient/tests/osc/v1/test_database_instances.py
@@ -404,14 +404,15 @@ class TestDatabaseInstanceResizeFlavor(TestInstances):
@mock.patch.object(utils, 'find_resource')
def test_instance_resize_flavor(self, mock_find):
args = ['instance1', 'flavor_id']
- mock_find.return_value = 'fake_instance_id'
-
+ mock_find.side_effect = [
+ mock.MagicMock(id='fake_instance_id'),
+ mock.MagicMock(id='fake_flavor_id')
+ ]
parsed_args = self.check_parser(self.cmd, args, [])
- result = self.cmd.take_action(parsed_args)
+ self.cmd.take_action(parsed_args)
self.instance_client.resize_instance.assert_called_with(
- 'fake_instance_id', 'flavor_id')
- self.assertIsNone(result)
+ 'fake_instance_id', 'fake_flavor_id')
class TestDatabaseInstanceUpgrade(TestInstances):