Cinder: Adds a api mv 3.66 check for snapshot creation

auniyal61 · auniyal61 · commit 896a1673540d · 2026-05-08T12:37:06.000+05:30
Since api mv 3.66 the --force parameter is not requied for snapshot creation. In-use volumes snapshot can be taken by default without requiring the force flag. This is already handled in cinder side. fixed conflict master -> 2026.1: added import: from openstack import utils as sdk_utils Closes-Bug: #2089188 Change-Id: I452ad973c76fa7f039797e8ea02556bf9fe7315c Signed-off-by: Amit Uniyal <auniyal@redhat.com> (cherry picked from commit ede91ec) (cherry picked from commit 97d6641)
diff --git a/openstackclient/tests/unit/volume/v3/test_volume_snapshot.py b/openstackclient/tests/unit/volume/v3/test_volume_snapshot.py
@@ -180,6 +180,59 @@ def test_snapshot_create_with_remote_source(self):
         )
         self.volume_sdk_client.create_snapshot.assert_not_called()
 
+    def test_snapshot_create_pre_v366(self):
+        self.set_volume_api_version('3.65')
+
+        arglist = ["--force", self.snapshot.name]
+        verifylist = [("force", True), ("snapshot_name", self.snapshot.name)]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        self.cmd.take_action(parsed_args)
+
+        # force parameter should be passed
+        self.volume_sdk_client.create_snapshot.assert_called_with(
+            volume_id=self.snapshot.volume_id,
+            force=True,
+            name=self.snapshot.name,
+            description=None,
+            metadata=None,
+        )
+
+    def test_snapshot_create_v366_or_later(self):
+        self.set_volume_api_version('3.66')
+
+        arglist = [self.snapshot.name]
+        verifylist = [("force", False), ("snapshot_name", self.snapshot.name)]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        self.cmd.take_action(parsed_args)
+
+        # force parameter should not be passed, for >=3.66
+        self.volume_sdk_client.create_snapshot.assert_called_with(
+            volume_id=self.snapshot.volume_id,
+            name=self.snapshot.name,
+            description=None,
+            metadata=None,
+        )
+
+    def test_snapshot_create_v366_or_later_with_force(self):
+        """--force should be ignored for microversion >= 3.66."""
+        self.set_volume_api_version('3.66')
+
+        arglist = ["--force", self.snapshot.name]
+        verifylist = [("force", True), ("snapshot_name", self.snapshot.name)]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        self.cmd.take_action(parsed_args)
+
+        # passed but ignored
+        self.volume_sdk_client.create_snapshot.assert_called_with(
+            volume_id=self.snapshot.volume_id,
+            name=self.snapshot.name,
+            description=None,
+            metadata=None,
+        )
+
 
 class TestVolumeSnapshotDelete(volume_fakes.TestVolume):
     def setUp(self):
diff --git a/openstackclient/volume/v3/volume_snapshot.py b/openstackclient/volume/v3/volume_snapshot.py
@@ -20,6 +20,7 @@
 
 from cliff import columns as cliff_columns
 from openstack.block_storage.v3 import snapshot as _snapshot
+from openstack import utils as sdk_utils
 from osc_lib.cli import format_columns
 from osc_lib.cli import parseractions
 from osc_lib.command import command
@@ -123,7 +124,9 @@ def get_parser(self, prog_name):
             action="store_true",
             default=False,
             help=_(
-                "Create a snapshot attached to an instance. Default is False"
+                "Allow snapshot of in-use (attached) volume. "
+                "Only needed for microversions prior to 3.66; "
+                "ignored for 3.66+"
             ),
         )
         parser.add_argument(
@@ -176,13 +179,23 @@ def take_action(self, parsed_args):
             )
         else:
             # Create a new snapshot from scratch
-            snapshot = volume_client.create_snapshot(
-                volume_id=volume_id,
-                force=parsed_args.force,
-                name=parsed_args.snapshot_name,
-                description=parsed_args.description,
-                metadata=parsed_args.properties,
-            )
+            # only for microversion < 3.66, pass force parameter
+            # for backward compatibility
+            if not sdk_utils.supports_microversion(volume_client, '3.66'):
+                snapshot = volume_client.create_snapshot(
+                    volume_id=volume_id,
+                    force=parsed_args.force,
+                    name=parsed_args.snapshot_name,
+                    description=parsed_args.description,
+                    metadata=parsed_args.properties,
+                )
+            else:
+                snapshot = volume_client.create_snapshot(
+                    volume_id=volume_id,
+                    name=parsed_args.snapshot_name,
+                    description=parsed_args.description,
+                    metadata=parsed_args.properties,
+                )
 
         data = _format_snapshot(snapshot)
         return zip(*sorted(data.items()))
diff --git a/releasenotes/notes/bug-2089188-snapshot-mv-366-force.yaml b/releasenotes/notes/bug-2089188-snapshot-mv-366-force.yaml
@@ -0,0 +1,9 @@
+---
+fixes:
+  - |
+    Fix ``volume snapshot create`` to work with Cinder API microversion 3.66
+    and later. Since version 3.66, the 'force' parameter is no longer
+    needed for snapshot creation as in-use volumes can be snapshotted by
+    default. The command now only passes the 'force' parameter for version
+    prior to 3.66.
+    [Bug `2089188 <https://bugs.launchpad.net/python-openstackclient/+bug/2089188>`_]
