openstack
/
cinder
Code Issues Proposed changes

Correctly report multiattach in Pure drivers 

As-is we end up ‘reporting’ to the scheduler that Pure backends do not
support multiattach. This is confusing because volumes created on a
Pure FlashArray will always be capable of doing multiattach even if not
specified by Cinder.

This change syncs up the reporting to the scheduler and what is actually
happening on the array.

Change-Id: I3a1e519efdd0c27f63b049aa686ff9045c260ab7
Closes-Bug: #1493478
This commit is contained in:
Patrick East 2015-09-08 11:06:36 -07:00
parent cb790c0979
commit 327df72aec
2 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
cinder/tests/unit/test_pure.py
View File

@ -547,6 +547,7 @@ class PureBaseVolumeDriverTestCase(PureDriverTestCase):
USED_SPACE),
"total_volumes": 100,
"filter_function": filter_function,
"multiattach": True,
}
real_result = self.driver.get_volume_stats(refresh=True)
self.assertDictMatch(result, real_result)
@ -574,6 +575,7 @@ class PureBaseVolumeDriverTestCase(PureDriverTestCase):
"max_over_subscription_ratio": DEFAULT_OVER_SUBSCRIPTION,
"total_volumes": 100,
"filter_function": filter_function,
"multiattach": True,
}
real_result = self.driver.get_volume_stats(refresh=True)
self.assertDictMatch(result, real_result)
@ -602,6 +604,7 @@ class PureBaseVolumeDriverTestCase(PureDriverTestCase):
"max_over_subscription_ratio": DEFAULT_OVER_SUBSCRIPTION,
"total_volumes": 0,
"filter_function": filter_function,
"multiattach": True,
}
real_result = self.driver.get_volume_stats(refresh=True)
self.assertDictMatch(result, real_result)

1
cinder/volume/drivers/pure.py
View File

@ -315,6 +315,7 @@ class PureBaseVolumeDriver(san.SanDriver):
"max_over_subscription_ratio": thin_provisioning,
"total_volumes": total_vols,
"filter_function": self.get_filter_function(),
"multiattach": True,
}
self._stats = data