Merge "volume: Add missing 'volume list --offset' parameter"

Author: Zuul

openstackclient/tests/unit/volume/v1/test_volume.py

@@ -858,16 +858,18 @@ def test_volume_list_long(self):
         ), )
         self.assertCountEqual(datalist, tuple(data))
 
-    def test_volume_list_with_limit(self):
+    def test_volume_list_with_limit_and_offset(self):
         arglist = [
             '--limit', '2',
+            '--offset', '5',
         ]
         verifylist = [
             ('long', False),
             ('all_projects', False),
             ('name', None),
             ('status', None),
             ('limit', 2),
+            ('offset', 5),
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
 
@@ -876,9 +878,11 @@ def test_volume_list_with_limit(self):
         self.volumes_mock.list.assert_called_once_with(
             limit=2,
             search_opts={
+                'offset': 5,
                 'status': None,
                 'display_name': None,
-                'all_tenants': False, }
+                'all_tenants': False,
+            },
         )
         self.assertEqual(self.columns, columns)
         self.assertCountEqual(self.datalist, tuple(data))

openstackclient/volume/v1/volume.py

@@ -327,6 +327,13 @@ def get_parser(self, prog_name):
             default=False,
             help=_('List additional fields in output'),
         )
+        parser.add_argument(
+            '--offset',
+            type=int,
+            action=parseractions.NonNegativeAction,
+            metavar='<offset>',
+            help=_('Index from which to start listing volumes'),
+        )
         parser.add_argument(
             '--limit',
             type=int,
@@ -395,6 +402,9 @@ def take_action(self, parsed_args):
             'status': parsed_args.status,
         }
 
+        if parsed_args.offset:
+            search_opts['offset'] = parsed_args.offset
+
         data = volume_client.volumes.list(
             search_opts=search_opts,
             limit=parsed_args.limit,