From 1a45a8d852c3c26c6aab15704628c9c8264348ad Mon Sep 17 00:00:00 2001 From: Clinton Knight Date: Mon, 11 Jan 2016 16:21:47 -0500 Subject: [PATCH] Remove invalid NetApp QoS keys The NetApp driver code contains two QoS keys that don't work. They aren't documented anywhere, and they are redundant, so the simplest fix is to remove them. Closes-Bug: #1532965 Change-Id: I21a638dc56cb779fbad54c4fe62ec9711a6b8098 --- cinder/volume/drivers/netapp/utils.py | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/cinder/volume/drivers/netapp/utils.py b/cinder/volume/drivers/netapp/utils.py index 0c1c7c0fc..a5585a576 100644 --- a/cinder/volume/drivers/netapp/utils.py +++ b/cinder/volume/drivers/netapp/utils.py @@ -50,8 +50,7 @@ DEPRECATED_SSC_SPECS = {'netapp_unmirrored': 'netapp_mirrored', 'netapp_nodedup': 'netapp_dedup', 'netapp_nocompression': 'netapp_compression', 'netapp_thick_provisioned': 'netapp_thin_provisioned'} -QOS_KEYS = frozenset( - ['maxIOPS', 'total_iops_sec', 'maxBPS', 'total_bytes_sec']) +QOS_KEYS = frozenset(['maxIOPS', 'maxBPS']) BACKEND_QOS_CONSUMERS = frozenset(['back-end', 'both']) @@ -204,9 +203,9 @@ def map_qos_spec(qos_spec, volume): spec = dict(policy_name=get_qos_policy_group_name(volume), max_throughput=None) # IOPS and BPS specifications are exclusive of one another. - if 'maxiops' in qos_spec or 'total_iops_sec' in qos_spec: + if 'maxiops' in qos_spec: spec['max_throughput'] = '%siops' % qos_spec['maxiops'] - elif 'maxbps' in qos_spec or 'total_bytes_sec' in qos_spec: + elif 'maxbps' in qos_spec: spec['max_throughput'] = '%sB/s' % qos_spec['maxbps'] return spec -- 2.45.2