summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSean McGinnis <sean.mcginnis@gmail.com>2018-07-26 11:41:57 -0500
committerSean McGinnis <sean.mcginnis@gmail.com>2018-07-26 11:41:57 -0500
commit32251f0ea3863098b4d4d54364c8ee18ff170a44 (patch)
treea018e13bfeb48a4a9fff4fce3fdd1f42bf0571ae
parent8d566689001a442c2312e366acc167af88888fd3 (diff)
downloadpython-cinderclient-32251f0ea3863098b4d4d54364c8ee18ff170a44.tar.gz
Remove replication v1 support
The replication v1 implementation in Cinder was deprecated in the Mitaka release in favor of the v2 "Cheesecake" version. Support was kept in the client for backwards compatibility, but it has now been several releases and these options should be removed. Closes-bug: #1705470 Change-Id: I978a39a552fffc9ac7ba6e4726d1df2072fa45ba Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
-rw-r--r--cinderclient/v2/shell.py7
-rw-r--r--cinderclient/v2/volumes.py3
-rw-r--r--cinderclient/v3/shell.py5
-rw-r--r--cinderclient/v3/volumes.py3
-rw-r--r--releasenotes/notes/remove-replv1-cabf2194edb9d963.yaml7
5 files changed, 10 insertions, 15 deletions
diff --git a/cinderclient/v2/shell.py b/cinderclient/v2/shell.py
index 16412d8..88c79d4 100644
--- a/cinderclient/v2/shell.py
+++ b/cinderclient/v2/shell.py
@@ -208,7 +208,7 @@ def do_show(cs, args):
class CheckSizeArgForCreate(argparse.Action):
def __call__(self, parser, args, values, option_string=None):
- if ((args.snapshot_id or args.source_volid or args.source_replica)
+ if ((args.snapshot_id or args.source_volid)
is None and values is None):
if not hasattr(args, 'backup_id') or args.backup_id is None:
parser.error('Size is a required parameter if snapshot '
@@ -240,10 +240,6 @@ class CheckSizeArgForCreate(argparse.Action):
help='Creates volume from volume ID. Default=None.')
@utils.arg('--source_volid',
help=argparse.SUPPRESS)
-@utils.arg('--source-replica',
- metavar='<source-replica>',
- default=None,
- help='Creates volume from replicated volume ID. Default=None.')
@utils.arg('--image-id',
metavar='<image-id>',
default=None,
@@ -343,7 +339,6 @@ def do_create(cs, args):
imageRef=image_ref,
metadata=volume_metadata,
scheduler_hints=hints,
- source_replica=args.source_replica,
multiattach=args.multiattach)
info = dict()
diff --git a/cinderclient/v2/volumes.py b/cinderclient/v2/volumes.py
index d8365df..18a6e25 100644
--- a/cinderclient/v2/volumes.py
+++ b/cinderclient/v2/volumes.py
@@ -242,7 +242,7 @@ class VolumeManager(base.ManagerWithFind):
volume_type=None, user_id=None,
project_id=None, availability_zone=None,
metadata=None, imageRef=None, scheduler_hints=None,
- source_replica=None, multiattach=False):
+ multiattach=False):
"""Create a volume.
:param size: Size of volume in GB
@@ -257,7 +257,6 @@ class VolumeManager(base.ManagerWithFind):
:param metadata: Optional metadata to set on volume creation
:param imageRef: reference to an image stored in glance
:param source_volid: ID of source volume to clone from
- :param source_replica: ID of source volume to clone replica
:param scheduler_hints: (optional extension) arbitrary key-value pairs
specified by the client to help boot an instance
:param multiattach: Allow the volume to be attached to more than
diff --git a/cinderclient/v3/shell.py b/cinderclient/v3/shell.py
index 1d4e18a..e523782 100644
--- a/cinderclient/v3/shell.py
+++ b/cinderclient/v3/shell.py
@@ -505,10 +505,6 @@ def do_reset_state(cs, args):
help='Creates volume from volume ID. Default=None.')
@utils.arg('--source_volid',
help=argparse.SUPPRESS)
-@utils.arg('--source-replica',
- metavar='<source-replica>',
- default=None,
- help='Creates volume from replicated volume ID. Default=None.')
@utils.arg('--image-id',
metavar='<image-id>',
default=None,
@@ -625,7 +621,6 @@ def do_create(cs, args):
imageRef=image_ref,
metadata=volume_metadata,
scheduler_hints=hints,
- source_replica=args.source_replica,
multiattach=args.multiattach,
backup_id=backup_id)
diff --git a/cinderclient/v3/volumes.py b/cinderclient/v3/volumes.py
index 90cf066..33850a3 100644
--- a/cinderclient/v3/volumes.py
+++ b/cinderclient/v3/volumes.py
@@ -77,7 +77,7 @@ class VolumeManager(volumes.VolumeManager):
volume_type=None, user_id=None,
project_id=None, availability_zone=None,
metadata=None, imageRef=None, scheduler_hints=None,
- source_replica=None, multiattach=False, backup_id=None):
+ multiattach=False, backup_id=None):
"""Create a volume.
:param size: Size of volume in GB
@@ -93,7 +93,6 @@ class VolumeManager(volumes.VolumeManager):
:param metadata: Optional metadata to set on volume creation
:param imageRef: reference to an image stored in glance
:param source_volid: ID of source volume to clone from
- :param source_replica: ID of source volume to clone replica
:param scheduler_hints: (optional extension) arbitrary key-value pairs
specified by the client to help boot an instance
:param multiattach: Allow the volume to be attached to more than
diff --git a/releasenotes/notes/remove-replv1-cabf2194edb9d963.yaml b/releasenotes/notes/remove-replv1-cabf2194edb9d963.yaml
new file mode 100644
index 0000000..28ad296
--- /dev/null
+++ b/releasenotes/notes/remove-replv1-cabf2194edb9d963.yaml
@@ -0,0 +1,7 @@
+---
+upgrade:
+ - |
+ The volume creation argument ``--source-replica`` on the command line and
+ the ``source_replica`` kwarg for the ``create()`` call when using the
+ cinderclient library were for the replication v1 support that was removed
+ in the Mitaka release. These options have now been removed.