diff --git a/releasenotes/notes/add-datastore-version-show-to-osc-8c2035dbf6104a9f.yaml b/releasenotes/notes/add-datastore-version-show-to-osc-8c2035dbf6104a9f.yaml new file mode 100644 index 00000000..5c516c42 --- /dev/null +++ b/releasenotes/notes/add-datastore-version-show-to-osc-8c2035dbf6104a9f.yaml @@ -0,0 +1,4 @@ +--- +features: + - The command ``trove datastore-version-show`` is now available to use in + the python-openstackclient CLI as ``openstack datastore version show`` diff --git a/setup.cfg b/setup.cfg index 00143a04..e4ba0e17 100644 --- a/setup.cfg +++ b/setup.cfg @@ -48,6 +48,7 @@ openstack.database.v1 = datastore_list = troveclient.osc.v1.datastores:ListDatastores datastore_show = troveclient.osc.v1.datastores:ShowDatastore datastore_version_list = troveclient.osc.v1.datastores:ListDatastoreVersions + datastore_version_show = troveclient.osc.v1.datastores:ShowDatastoreVersion [build_sphinx] diff --git a/troveclient/osc/v1/datastores.py b/troveclient/osc/v1/datastores.py index d442a6c9..85983f0a 100644 --- a/troveclient/osc/v1/datastores.py +++ b/troveclient/osc/v1/datastores.py @@ -16,7 +16,9 @@ from osc_lib.command import command from osc_lib import utils import six +from troveclient import exceptions from troveclient.i18n import _ +from troveclient import utils as tc_utils def set_attributes_for_print_detail(datastore): @@ -86,3 +88,40 @@ class ListDatastoreVersions(command.Lister): versions = datastore_version_client.list(parsed_args.datastore) ds = [utils.get_item_properties(d, self.columns) for d in versions] return self.columns, ds + + +class ShowDatastoreVersion(command.ShowOne): + _description = _("Shows details of a datastore version.") + + def get_parser(self, prog_name): + parser = super(ShowDatastoreVersion, self).get_parser(prog_name) + parser.add_argument( + 'datastore_version', + metavar='', + help=_('ID or name of the datastore version.'), + ) + parser.add_argument( + '--datastore', + metavar='', + default=None, + help=_('ID or name of the datastore. Optional if the ID of' + 'the datastore_version is provided.'), + ) + return parser + + def take_action(self, parsed_args): + datastore_version_client =\ + self.app.client_manager.database.datastore_versions + if parsed_args.datastore: + datastore_version = datastore_version_client.\ + get(parsed_args.datastore, parsed_args.datastore_version) + elif tc_utils.is_uuid_like(parsed_args.datastore_version): + datastore_version = datastore_version_client.\ + get_by_uuid(parsed_args.datastore_version) + else: + raise exceptions.NoUniqueMatch(_('The datastore name or id is' + ' required to retrieve a' + ' datastore version by name.')) + if datastore_version._info.get('links'): + del(datastore_version._info['links']) + return zip(*sorted(six.iteritems(datastore_version._info))) diff --git a/troveclient/tests/osc/v1/test_datastores.py b/troveclient/tests/osc/v1/test_datastores.py index 4a586670..c9e58068 100644 --- a/troveclient/tests/osc/v1/test_datastores.py +++ b/troveclient/tests/osc/v1/test_datastores.py @@ -11,6 +11,7 @@ # under the License. from troveclient import common +from troveclient import exceptions from troveclient.osc.v1 import datastores from troveclient.tests.osc.v1 import fakes @@ -85,3 +86,40 @@ class TestDatastoreVersionList(TestDatastores): self.datastore_version_client.list.assert_called_once_with(args[0]) self.assertEqual(self.columns, columns) self.assertEqual([self.values], data) + + +class TestDatastoreVersionShow(TestDatastores): + values = ('v-56', '5.6') + + def setUp(self): + super(TestDatastoreVersionShow, self).setUp() + self.cmd = datastores.ShowDatastoreVersion(self.app, None) + self.data = self.fake_datastores.get_datastores_d_123_versions() + self.datastore_version_client.get.return_value = self.data + self.columns = ( + 'id', + 'name', + ) + + def test_datastore_version_show_defaults(self): + args = ['5.6', '--datastore', 'mysql'] + verifylist = [ + ('datastore_version', '5.6'), + ('datastore', 'mysql'), + ] + parsed_args = self.check_parser(self.cmd, args, verifylist) + columns, data = self.cmd.take_action(parsed_args) + self.assertEqual(self.columns, columns) + self.assertEqual(self.values, data) + + def test_datastore_version_show_with_version_id_exception(self): + args = [ + 'v-56', + ] + verifylist = [ + ('datastore_version', 'v-56'), + ] + parsed_args = self.check_parser(self.cmd, args, verifylist) + self.assertRaises(exceptions.NoUniqueMatch, + self.cmd.take_action, + parsed_args)