From 8d5a563aba6f6d66d8b240812ba4155fb3f03ff0 Mon Sep 17 00:00:00 2001 From: Simon Dodsley Date: Fri, 21 Oct 2016 16:35:26 -0400 Subject: [PATCH] Missing parameter for storage pools in Pure Storage driver Storage pools requires a manadatory parameter called QoS_support. This is missing in the Pure Storage driver so we will add it with the value of False Also add parameter to unit test. Fixes bug 1635342 Change-Id: I9f81f72fddf85ed7dee920ae48bea7f9707d4744 --- cinder/tests/unit/volume/drivers/test_pure.py | 1 + cinder/volume/drivers/pure.py | 1 + 2 files changed, 2 insertions(+) diff --git a/cinder/tests/unit/volume/drivers/test_pure.py b/cinder/tests/unit/volume/drivers/test_pure.py index 369d7f23f03..7af8c1208ea 100644 --- a/cinder/tests/unit/volume/drivers/test_pure.py +++ b/cinder/tests/unit/volume/drivers/test_pure.py @@ -2694,6 +2694,7 @@ class PureVolumeUpdateStatsTestCase(PureBaseSharedDriverTestCase): 'consistencygroup_support': True, 'thin_provisioning_support': True, 'multiattach': True, + 'QoS_support': False, 'total_capacity_gb': TOTAL_CAPACITY, 'free_capacity_gb': TOTAL_CAPACITY - USED_SPACE, 'reserved_percentage': reserved_percentage, diff --git a/cinder/volume/drivers/pure.py b/cinder/volume/drivers/pure.py index 5826d58bf3f..5c03fce2cbc 100644 --- a/cinder/volume/drivers/pure.py +++ b/cinder/volume/drivers/pure.py @@ -556,6 +556,7 @@ class PureBaseVolumeDriver(san.SanDriver): data['consistencygroup_support'] = True data['thin_provisioning_support'] = True data['multiattach'] = True + data['QoS_support'] = False # Add capacity info for scheduler data['total_capacity_gb'] = total_capacity