python-openstackclient 9.0.0__py3-none-any.whl → 10.0.0__py3-none-any.whl

openstackclient/volume/client.py

@@ -13,7 +13,9 @@
 #   under the License.
 #
 
+import argparse
 import logging
+from typing import Any
 
 from osc_lib import exceptions
 from osc_lib import utils
@@ -34,7 +36,7 @@ API_VERSIONS = {
 _volume_api_version = None
 
 
-def make_client(instance):
+def make_client(instance: Any) -> Any:
     """Returns a volume service client."""
 
     # Defer client imports until we actually need them
@@ -92,7 +94,9 @@ def make_client(instance):
     return client
 
 
-def build_option_parser(parser):
+def build_option_parser(
+    parser: argparse.ArgumentParser,
+) -> argparse.ArgumentParser:
     """Hook to add global options"""
     parser.add_argument(
         '--os-volume-api-version',
@@ -104,7 +108,7 @@ def build_option_parser(parser):
     return parser
 
 
-def check_api_version(check_version):
+def check_api_version(check_version: str) -> bool:
     """Validate version supplied by user
 
     Returns:

openstackclient/volume/v2/backup_record.py

@@ -14,7 +14,10 @@
 
 """Volume v2 Backup action implementations"""
 
+import argparse
 import logging
+from collections.abc import Iterable, Sequence
+from typing import Any
 
 from osc_lib import utils
 
@@ -33,7 +36,7 @@ Backup information can be imported into a new service instance to be able to
 restore."""
     )
 
-    def get_parser(self, prog_name):
+    def get_parser(self, prog_name: str) -> argparse.ArgumentParser:
         parser = super().get_parser(prog_name)
         parser.add_argument(
             "backup",
@@ -42,7 +45,9 @@ restore."""
         )
         return parser
 
-    def take_action(self, parsed_args):
+    def take_action(
+        self, parsed_args: argparse.Namespace
+    ) -> tuple[Sequence[str], Iterable[Any]]:
         volume_client = self.app.client_manager.volume
         backup = utils.find_resource(volume_client.backups, parsed_args.backup)
         backup_data = volume_client.backups.export_record(backup.id)
@@ -53,7 +58,8 @@ restore."""
             backup_data['Backup Service'] = backup_data.pop('backup_service')
             backup_data['Metadata'] = backup_data.pop('backup_url')
 
-        return zip(*sorted(backup_data.items()))
+        col_headers, col_data = zip(*sorted(backup_data.items()))
+        return col_headers, col_data
 
 
 class ImportBackupRecord(command.ShowOne):
@@ -64,7 +70,7 @@ Exported backup details contain the metadata necessary to restore to a new or
 rebuilt service instance"""
     )
 
-    def get_parser(self, prog_name):
+    def get_parser(self, prog_name: str) -> argparse.ArgumentParser:
         parser = super().get_parser(prog_name)
         parser.add_argument(
             "backup_service",
@@ -78,10 +84,13 @@ rebuilt service instance"""
         )
         return parser
 
-    def take_action(self, parsed_args):
+    def take_action(
+        self, parsed_args: argparse.Namespace
+    ) -> tuple[Sequence[str], Iterable[Any]]:
         volume_client = self.app.client_manager.volume
         backup_data = volume_client.backups.import_record(
             parsed_args.backup_service, parsed_args.backup_metadata
         )
         backup_data.pop('links', None)
-        return zip(*sorted(backup_data.items()))
+        col_headers, col_data = zip(*sorted(backup_data.items()))
+        return col_headers, col_data

openstackclient/volume/v2/consistency_group.py

@@ -16,6 +16,8 @@
 
 import argparse
 import logging
+from collections.abc import Iterable, Sequence
+from typing import Any
 
 from osc_lib.cli import format_columns
 from osc_lib import exceptions
@@ -28,7 +30,9 @@ from openstackclient.i18n import _
 LOG = logging.getLogger(__name__)
 
 
-def _find_volumes(parsed_args_volumes, volume_client):
+def _find_volumes(
+    parsed_args_volumes: list[str], volume_client: Any
+) -> tuple[int, str]:
     result = 0
     uuid = ''
     for volume in parsed_args_volumes:
@@ -48,7 +52,7 @@ def _find_volumes(parsed_args_volumes, volume_client):
 class AddVolumeToConsistencyGroup(command.Command):
     _description = _("Add volume(s) to consistency group")
 
-    def get_parser(self, prog_name):
+    def get_parser(self, prog_name: str) -> argparse.ArgumentParser:
         parser = super().get_parser(prog_name)
         parser.add_argument(
             'consistency_group',
@@ -66,7 +70,7 @@ class AddVolumeToConsistencyGroup(command.Command):
         )
         return parser
 
-    def take_action(self, parsed_args):
+    def take_action(self, parsed_args: argparse.Namespace) -> None:
         volume_client = self.app.client_manager.volume
         result, add_uuid = _find_volumes(parsed_args.volumes, volume_client)
 
@@ -90,7 +94,7 @@ class AddVolumeToConsistencyGroup(command.Command):
 class CreateConsistencyGroup(command.ShowOne):
     _description = _("Create new consistency group.")
 
-    def get_parser(self, prog_name):
+    def get_parser(self, prog_name: str) -> argparse.ArgumentParser:
         parser = super().get_parser(prog_name)
         parser.add_argument(
             "name",
@@ -144,7 +148,9 @@ class CreateConsistencyGroup(command.ShowOne):
         )
         return parser
 
-    def take_action(self, parsed_args):
+    def take_action(
+        self, parsed_args: argparse.Namespace
+    ) -> tuple[Sequence[str], Iterable[Any]]:
         volume_client = self.app.client_manager.volume
         if parsed_args.volume_type:
             volume_type_id = utils.find_resource(
@@ -186,13 +192,14 @@ class CreateConsistencyGroup(command.ShowOne):
                 )
             )
 
-        return zip(*sorted(consistency_group._info.items()))
+        col_headers, col_data = zip(*sorted(consistency_group._info.items()))
+        return col_headers, col_data
 
 
 class DeleteConsistencyGroup(command.Command):
     _description = _("Delete consistency group(s).")
 
-    def get_parser(self, prog_name):
+    def get_parser(self, prog_name: str) -> argparse.ArgumentParser:
         parser = super().get_parser(prog_name)
         parser.add_argument(
             'consistency_groups',
@@ -208,7 +215,7 @@ class DeleteConsistencyGroup(command.Command):
         )
         return parser
 
-    def take_action(self, parsed_args):
+    def take_action(self, parsed_args: argparse.Namespace) -> None:
         volume_client = self.app.client_manager.volume
         result = 0
 
@@ -241,7 +248,7 @@ class DeleteConsistencyGroup(command.Command):
 class ListConsistencyGroup(command.Lister):
     _description = _("List consistency groups.")
 
-    def get_parser(self, prog_name):
+    def get_parser(self, prog_name: str) -> argparse.ArgumentParser:
         parser = super().get_parser(prog_name)
         parser.add_argument(
             '--all-projects',
@@ -258,7 +265,9 @@ class ListConsistencyGroup(command.Lister):
         )
         return parser
 
-    def take_action(self, parsed_args):
+    def take_action(
+        self, parsed_args: argparse.Namespace
+    ) -> tuple[Sequence[str], Iterable[tuple[Any, ...]]]:
         if parsed_args.long:
             columns = [
                 'ID',
@@ -292,7 +301,7 @@ class ListConsistencyGroup(command.Lister):
 class RemoveVolumeFromConsistencyGroup(command.Command):
     _description = _("Remove volume(s) from consistency group")
 
-    def get_parser(self, prog_name):
+    def get_parser(self, prog_name: str) -> argparse.ArgumentParser:
         parser = super().get_parser(prog_name)
         parser.add_argument(
             'consistency_group',
@@ -310,7 +319,7 @@ class RemoveVolumeFromConsistencyGroup(command.Command):
         )
         return parser
 
-    def take_action(self, parsed_args):
+    def take_action(self, parsed_args: argparse.Namespace) -> None:
         volume_client = self.app.client_manager.volume
         result, remove_uuid = _find_volumes(parsed_args.volumes, volume_client)
 
@@ -334,7 +343,7 @@ class RemoveVolumeFromConsistencyGroup(command.Command):
 class SetConsistencyGroup(command.Command):
     _description = _("Set consistency group properties")
 
-    def get_parser(self, prog_name):
+    def get_parser(self, prog_name: str) -> argparse.ArgumentParser:
         parser = super().get_parser(prog_name)
         parser.add_argument(
             'consistency_group',
@@ -353,7 +362,7 @@ class SetConsistencyGroup(command.Command):
         )
         return parser
 
-    def take_action(self, parsed_args):
+    def take_action(self, parsed_args: argparse.Namespace) -> None:
         volume_client = self.app.client_manager.volume
         kwargs = {}
         if parsed_args.name:
@@ -372,7 +381,7 @@ class SetConsistencyGroup(command.Command):
 class ShowConsistencyGroup(command.ShowOne):
     _description = _("Display consistency group details.")
 
-    def get_parser(self, prog_name):
+    def get_parser(self, prog_name: str) -> argparse.ArgumentParser:
         parser = super().get_parser(prog_name)
         parser.add_argument(
             "consistency_group",
@@ -381,9 +390,12 @@ class ShowConsistencyGroup(command.ShowOne):
         )
         return parser
 
-    def take_action(self, parsed_args):
+    def take_action(
+        self, parsed_args: argparse.Namespace
+    ) -> tuple[Sequence[str], Iterable[Any]]:
         volume_client = self.app.client_manager.volume
         consistency_group = utils.find_resource(
             volume_client.consistencygroups, parsed_args.consistency_group
         )
-        return zip(*sorted(consistency_group._info.items()))
+        col_headers, col_data = zip(*sorted(consistency_group._info.items()))
+        return col_headers, col_data

openstackclient/volume/v2/consistency_group_snapshot.py

@@ -14,7 +14,10 @@
 
 """Volume v2 consistency group snapshot action implementations"""
 
+import argparse
 import logging
+from collections.abc import Iterable, Sequence
+from typing import Any
 
 from osc_lib import exceptions
 from osc_lib import utils
@@ -29,7 +32,7 @@ LOG = logging.getLogger(__name__)
 class CreateConsistencyGroupSnapshot(command.ShowOne):
     _description = _("Create new consistency group snapshot.")
 
-    def get_parser(self, prog_name):
+    def get_parser(self, prog_name: str) -> argparse.ArgumentParser:
         parser = super().get_parser(prog_name)
         parser.add_argument(
             "snapshot_name",
@@ -52,7 +55,9 @@ class CreateConsistencyGroupSnapshot(command.ShowOne):
         )
         return parser
 
-    def take_action(self, parsed_args):
+    def take_action(
+        self, parsed_args: argparse.Namespace
+    ) -> tuple[Sequence[str], Iterable[Any]]:
         volume_client = self.app.client_manager.volume
         consistency_group = parsed_args.consistency_group
         if not parsed_args.consistency_group:
@@ -68,13 +73,16 @@ class CreateConsistencyGroupSnapshot(command.ShowOne):
             description=parsed_args.description,
         )
 
-        return zip(*sorted(consistency_group_snapshot._info.items()))
+        col_headers, col_data = zip(
+            *sorted(consistency_group_snapshot._info.items())
+        )
+        return col_headers, col_data
 
 
 class DeleteConsistencyGroupSnapshot(command.Command):
     _description = _("Delete consistency group snapshot(s).")
 
-    def get_parser(self, prog_name):
+    def get_parser(self, prog_name: str) -> argparse.ArgumentParser:
         parser = super().get_parser(prog_name)
         parser.add_argument(
             "consistency_group_snapshot",
@@ -84,7 +92,7 @@ class DeleteConsistencyGroupSnapshot(command.Command):
         )
         return parser
 
-    def take_action(self, parsed_args):
+    def take_action(self, parsed_args: argparse.Namespace) -> None:
         volume_client = self.app.client_manager.volume
         result = 0
 
@@ -117,7 +125,7 @@ class DeleteConsistencyGroupSnapshot(command.Command):
 class ListConsistencyGroupSnapshot(command.Lister):
     _description = _("List consistency group snapshots.")
 
-    def get_parser(self, prog_name):
+    def get_parser(self, prog_name: str) -> argparse.ArgumentParser:
         parser = super().get_parser(prog_name)
         parser.add_argument(
             '--all-projects',
@@ -153,7 +161,9 @@ class ListConsistencyGroupSnapshot(command.Lister):
         )
         return parser
 
-    def take_action(self, parsed_args):
+    def take_action(
+        self, parsed_args: argparse.Namespace
+    ) -> tuple[Sequence[str], Iterable[tuple[Any, ...]]]:
         if parsed_args.long:
             columns = [
                 'ID',
@@ -194,7 +204,7 @@ class ListConsistencyGroupSnapshot(command.Lister):
 class ShowConsistencyGroupSnapshot(command.ShowOne):
     _description = _("Display consistency group snapshot details")
 
-    def get_parser(self, prog_name):
+    def get_parser(self, prog_name: str) -> argparse.ArgumentParser:
         parser = super().get_parser(prog_name)
         parser.add_argument(
             "consistency_group_snapshot",
@@ -203,9 +213,14 @@ class ShowConsistencyGroupSnapshot(command.ShowOne):
         )
         return parser
 
-    def take_action(self, parsed_args):
+    def take_action(
+        self, parsed_args: argparse.Namespace
+    ) -> tuple[Sequence[str], Iterable[Any]]:
         volume_client = self.app.client_manager.volume
         consistency_group_snapshot = utils.find_resource(
             volume_client.cgsnapshots, parsed_args.consistency_group_snapshot
         )
-        return zip(*sorted(consistency_group_snapshot._info.items()))
+        col_headers, col_data = zip(
+            *sorted(consistency_group_snapshot._info.items())
+        )
+        return col_headers, col_data

openstackclient/volume/v2/qos_specs.py

@@ -15,7 +15,10 @@
 
 """Volume v2 QoS action implementations"""
 
+import argparse
 import logging
+from collections.abc import Iterable, Sequence
+from typing import Any
 
 from osc_lib.cli import format_columns
 from osc_lib.cli import parseractions
@@ -32,7 +35,7 @@ LOG = logging.getLogger(__name__)
 class AssociateQos(command.Command):
     _description = _("Associate a QoS specification to a volume type")
 
-    def get_parser(self, prog_name):
+    def get_parser(self, prog_name: str) -> argparse.ArgumentParser:
         parser = super().get_parser(prog_name)
         parser.add_argument(
             'qos_spec',
@@ -46,7 +49,7 @@ class AssociateQos(command.Command):
         )
         return parser
 
-    def take_action(self, parsed_args):
+    def take_action(self, parsed_args: argparse.Namespace) -> None:
         volume_client = self.app.client_manager.volume
         qos_spec = utils.find_resource(
             volume_client.qos_specs, parsed_args.qos_spec
@@ -61,7 +64,7 @@ class AssociateQos(command.Command):
 class CreateQos(command.ShowOne):
     _description = _("Create new QoS specification")
 
-    def get_parser(self, prog_name):
+    def get_parser(self, prog_name: str) -> argparse.ArgumentParser:
         parser = super().get_parser(prog_name)
         parser.add_argument(
             'name',
@@ -93,7 +96,9 @@ class CreateQos(command.ShowOne):
         )
         return parser
 
-    def take_action(self, parsed_args):
+    def take_action(
+        self, parsed_args: argparse.Namespace
+    ) -> tuple[Sequence[str], Iterable[Any]]:
         volume_client = self.app.client_manager.volume
         specs = {}
         specs.update({'consumer': parsed_args.consumer})
@@ -110,13 +115,14 @@ class CreateQos(command.ShowOne):
                 )
             }
         )
-        return zip(*sorted(qos_spec._info.items()))
+        col_headers, col_data = zip(*sorted(qos_spec._info.items()))
+        return col_headers, col_data
 
 
 class DeleteQos(command.Command):
     _description = _("Delete QoS specification")
 
-    def get_parser(self, prog_name):
+    def get_parser(self, prog_name: str) -> argparse.ArgumentParser:
         parser = super().get_parser(prog_name)
         parser.add_argument(
             'qos_specs',
@@ -132,7 +138,7 @@ class DeleteQos(command.Command):
         )
         return parser
 
-    def take_action(self, parsed_args):
+    def take_action(self, parsed_args: argparse.Namespace) -> None:
         volume_client = self.app.client_manager.volume
         result = 0
 
@@ -161,7 +167,7 @@ class DeleteQos(command.Command):
 class DisassociateQos(command.Command):
     _description = _("Disassociate a QoS specification from a volume type")
 
-    def get_parser(self, prog_name):
+    def get_parser(self, prog_name: str) -> argparse.ArgumentParser:
         parser = super().get_parser(prog_name)
         parser.add_argument(
             'qos_spec',
@@ -183,7 +189,7 @@ class DisassociateQos(command.Command):
 
         return parser
 
-    def take_action(self, parsed_args):
+    def take_action(self, parsed_args: argparse.Namespace) -> None:
         volume_client = self.app.client_manager.volume
         qos_spec = utils.find_resource(
             volume_client.qos_specs, parsed_args.qos_spec
@@ -201,7 +207,9 @@ class DisassociateQos(command.Command):
 class ListQos(command.Lister):
     _description = _("List QoS specifications")
 
-    def take_action(self, parsed_args):
+    def take_action(
+        self, parsed_args: argparse.Namespace
+    ) -> tuple[Sequence[str], Iterable[tuple[Any, ...]]]:
         volume_client = self.app.client_manager.volume
         qos_specs_list = volume_client.qos_specs.list()
 
@@ -249,7 +257,7 @@ class ListQos(command.Lister):
 class SetQos(command.Command):
     _description = _("Set QoS specification properties")
 
-    def get_parser(self, prog_name):
+    def get_parser(self, prog_name: str) -> argparse.ArgumentParser:
         parser = super().get_parser(prog_name)
         parser.add_argument(
             'qos_spec',
@@ -277,7 +285,7 @@ class SetQos(command.Command):
         )
         return parser
 
-    def take_action(self, parsed_args):
+    def take_action(self, parsed_args: argparse.Namespace) -> None:
         volume_client = self.app.client_manager.volume
         qos_spec = utils.find_resource(
             volume_client.qos_specs, parsed_args.qos_spec
@@ -311,7 +319,7 @@ class SetQos(command.Command):
 class ShowQos(command.ShowOne):
     _description = _("Display QoS specification details")
 
-    def get_parser(self, prog_name):
+    def get_parser(self, prog_name: str) -> argparse.ArgumentParser:
         parser = super().get_parser(prog_name)
         parser.add_argument(
             'qos_spec',
@@ -320,7 +328,9 @@ class ShowQos(command.ShowOne):
         )
         return parser
 
-    def take_action(self, parsed_args):
+    def take_action(
+        self, parsed_args: argparse.Namespace
+    ) -> tuple[Sequence[str], Iterable[Any]]:
         volume_client = self.app.client_manager.volume
         qos_spec = utils.find_resource(
             volume_client.qos_specs, parsed_args.qos_spec
@@ -342,13 +352,14 @@ class ShowQos(command.ShowOne):
             }
         )
 
-        return zip(*sorted(qos_spec._info.items()))
+        col_headers, col_data = zip(*sorted(qos_spec._info.items()))
+        return col_headers, col_data
 
 
 class UnsetQos(command.Command):
     _description = _("Unset QoS specification properties")
 
-    def get_parser(self, prog_name):
+    def get_parser(self, prog_name: str) -> argparse.ArgumentParser:
         parser = super().get_parser(prog_name)
         parser.add_argument(
             'qos_spec',
@@ -367,7 +378,7 @@ class UnsetQos(command.Command):
         )
         return parser
 
-    def take_action(self, parsed_args):
+    def take_action(self, parsed_args: argparse.Namespace) -> None:
         volume_client = self.app.client_manager.volume
         qos_spec = utils.find_resource(
             volume_client.qos_specs, parsed_args.qos_spec

openstackclient/volume/v2/service.py

@@ -14,6 +14,11 @@
 
 """Service action implementations"""
 
+import argparse
+from collections.abc import Iterable, Sequence
+from typing import Any
+
+from openstack import utils as sdk_utils
 from osc_lib import exceptions
 from osc_lib import utils
 
@@ -24,7 +29,7 @@ from openstackclient.i18n import _
 class ListService(command.Lister):
     _description = _("List service command")
 
-    def get_parser(self, prog_name):
+    def get_parser(self, prog_name: str) -> argparse.ArgumentParser:
         parser = super().get_parser(prog_name)
         parser.add_argument(
             "--host",
@@ -44,8 +49,12 @@ class ListService(command.Lister):
         )
         return parser
 
-    def take_action(self, parsed_args):
-        volume_client = self.app.client_manager.sdk_connection.volume
+    def take_action(
+        self, parsed_args: argparse.Namespace
+    ) -> tuple[Sequence[str], Iterable[tuple[Any, ...]]]:
+        volume_client = sdk_utils.ensure_service_version(
+            self.app.client_manager.sdk_connection.volume, '2'
+        )
 
         columns: tuple[str, ...] = (
             "binary",
@@ -86,7 +95,7 @@ class ListService(command.Lister):
 class SetService(command.Command):
     _description = _("Set volume service properties")
 
-    def get_parser(self, prog_name):
+    def get_parser(self, prog_name: str) -> argparse.ArgumentParser:
         parser = super().get_parser(prog_name)
         parser.add_argument(
             "host",
@@ -115,7 +124,7 @@ class SetService(command.Command):
         )
         return parser
 
-    def take_action(self, parsed_args):
+    def take_action(self, parsed_args: argparse.Namespace) -> None:
         if parsed_args.disable_reason and not parsed_args.disable:
             msg = _(
                 "Cannot specify option --disable-reason without "
@@ -123,7 +132,9 @@ class SetService(command.Command):
             )
             raise exceptions.CommandError(msg)
 
-        volume_client = self.app.client_manager.sdk_connection.volume
+        volume_client = sdk_utils.ensure_service_version(
+            self.app.client_manager.sdk_connection.volume, '2'
+        )
 
         service = volume_client.find_service(
             parsed_args.service, ignore_missing=False, host=parsed_args.host