Revert "Revert "exp: Renegenerate the iSCSI patch""
This reverts commit dccd9a00e2c4e53082718c2b7126d6ca9308443e.
Reason for revert: Upstream updated
Change-Id: I00d628c9272df27d8bddaa503124d5235c1549b6
diff --git a/patches/openstack/cinder/sep-sp-iscsi.patch b/patches/openstack/cinder/sep-sp-iscsi.patch
index 085e9e8..71e3356 100644
--- a/patches/openstack/cinder/sep-sp-iscsi.patch
+++ b/patches/openstack/cinder/sep-sp-iscsi.patch
@@ -1,4 +1,4 @@
-From 809a5c657faa1b1cbe26a7ae0aad77840300498f Mon Sep 17 00:00:00 2001
+From 06914652d6de93e3d14209ff6a78b930826f0823 Mon Sep 17 00:00:00 2001
From: Peter Penchev <openstack-dev@storpool.com>
Date: Mon, 12 Mar 2018 12:00:10 +0200
Subject: [PATCH 9/9] Add iSCSI export support to the StorPool driver
@@ -33,7 +33,7 @@
create mode 100644 releasenotes/notes/storpool-iscsi-cefcfe590a07c5c7.yaml
diff --git a/cinder/tests/unit/volume/drivers/test_storpool.py b/cinder/tests/unit/volume/drivers/test_storpool.py
-index 94074d218..781d1a0ef 100644
+index 2dbbe2861..32a5d5c45 100644
--- a/cinder/tests/unit/volume/drivers/test_storpool.py
+++ b/cinder/tests/unit/volume/drivers/test_storpool.py
@@ -14,14 +14,24 @@
@@ -94,7 +94,7 @@
class MockDisk(object):
def __init__(self, diskId):
self.id = diskId
-@@ -180,6 +203,273 @@ def MockVolumeUpdateDesc(size):
+@@ -194,6 +217,273 @@ def MockVolumeUpdateDesc(size):
return {'size': size}
@@ -368,7 +368,7 @@
def MockSPConfig(section = 's01'):
res = {}
m = re.match('^s0*([A-Za-z0-9]+)$', section)
-@@ -221,7 +511,15 @@ class StorPoolTestCase(test.TestCase):
+@@ -236,7 +526,15 @@ class StorPoolTestCase(test.TestCase):
self.cfg.volume_backend_name = 'storpool_test'
self.cfg.storpool_template = None
self.cfg.storpool_replication = 3
@@ -384,7 +384,7 @@
mock_exec = mock.Mock()
mock_exec.return_value = ('', '')
-@@ -230,7 +528,7 @@ class StorPoolTestCase(test.TestCase):
+@@ -245,7 +543,7 @@ class StorPoolTestCase(test.TestCase):
self.driver.check_for_setup_error()
@ddt.data(
@@ -393,7 +393,7 @@
({'no-host': None}, KeyError),
({'host': 'sbad'}, driver.StorPoolConfigurationInvalid),
({'host': 's01'}, None),
-@@ -246,7 +544,7 @@ class StorPoolTestCase(test.TestCase):
+@@ -261,7 +559,7 @@ class StorPoolTestCase(test.TestCase):
conn)
@ddt.data(
@@ -402,7 +402,7 @@
({'no-host': None}, KeyError),
({'host': 'sbad'}, driver.StorPoolConfigurationInvalid),
)
-@@ -285,7 +583,7 @@ class StorPoolTestCase(test.TestCase):
+@@ -300,7 +598,7 @@ class StorPoolTestCase(test.TestCase):
self.assertEqual(21, pool['total_capacity_gb'])
self.assertEqual(5, int(pool['free_capacity_gb']))
@@ -411,10 +411,10 @@
self.assertFalse(pool['QoS_support'])
self.assertFalse(pool['thick_provisioning_support'])
self.assertTrue(pool['thin_provisioning_support'])
-@@ -641,3 +939,136 @@ class StorPoolTestCase(test.TestCase):
- self.driver.get_pool({
+@@ -656,6 +954,139 @@ class StorPoolTestCase(test.TestCase):
'volume_type': volume_type
}))
+
+
+ @ddt.data(
+ # The default values
@@ -548,8 +548,11 @@
+ cfg_final.targets['1'].volume,
+ volumeName(fconst.VOLUME_ID),
+ )
+ def test_volume_revert(self):
+ vol_id = 'rev1'
+ vol_name = volumeName(vol_id)
diff --git a/cinder/volume/drivers/storpool.py b/cinder/volume/drivers/storpool.py
-index caa0ce06e..cec7589f2 100644
+index 0051442ed..cb3beaf0c 100644
--- a/cinder/volume/drivers/storpool.py
+++ b/cinder/volume/drivers/storpool.py
@@ -15,6 +15,7 @@
@@ -592,7 +595,7 @@
cfg.StrOpt('storpool_template',
default=None,
help='The StorPool template for volumes with no type.'),
-@@ -104,6 +130,7 @@ class StorPoolDriver(driver.VolumeDriver):
+@@ -105,6 +131,7 @@ class StorPoolDriver(driver.VolumeDriver):
self._ourId = None
self._ourIdInt = None
self._attach = None
@@ -600,7 +603,7 @@
@staticmethod
def get_driver_options():
-@@ -161,10 +188,326 @@ class StorPoolDriver(driver.VolumeDriver):
+@@ -162,10 +189,326 @@ class StorPoolDriver(driver.VolumeDriver):
raise StorPoolConfigurationInvalid(
section=hostname, param='SP_OURID', error=e)
@@ -927,7 +930,7 @@
return {'driver_volume_type': 'storpool',
'data': {
'client_id': self._storpool_client_id(connector),
-@@ -173,6 +516,9 @@ class StorPoolDriver(driver.VolumeDriver):
+@@ -174,6 +517,9 @@ class StorPoolDriver(driver.VolumeDriver):
}}
def terminate_connection(self, volume, connector, **kwargs):
@@ -937,7 +940,7 @@
pass
def create_snapshot(self, snapshot):
-@@ -274,11 +620,20 @@ class StorPoolDriver(driver.VolumeDriver):
+@@ -275,11 +621,20 @@ class StorPoolDriver(driver.VolumeDriver):
)
def create_export(self, context, volume, connector):
@@ -959,7 +962,7 @@
def delete_volume(self, volume):
name = self._attach.volumeName(volume['id'])
try:
-@@ -315,6 +670,17 @@ class StorPoolDriver(driver.VolumeDriver):
+@@ -316,6 +671,17 @@ class StorPoolDriver(driver.VolumeDriver):
LOG.error("StorPoolDriver API initialization failed: %s", e)
raise
@@ -977,7 +980,7 @@
def _update_volume_stats(self):
try:
dl = self._attach.api().disksList()
-@@ -340,7 +706,7 @@ class StorPoolDriver(driver.VolumeDriver):
+@@ -341,7 +707,7 @@ class StorPoolDriver(driver.VolumeDriver):
'total_capacity_gb': total / units.Gi,
'free_capacity_gb': free / units.Gi,
'reserved_percentage': 0,
@@ -986,7 +989,7 @@
'QoS_support': False,
'thick_provisioning_support': False,
'thin_provisioning_support': True,
-@@ -359,7 +725,9 @@ class StorPoolDriver(driver.VolumeDriver):
+@@ -360,7 +726,9 @@ class StorPoolDriver(driver.VolumeDriver):
'volume_backend_name') or 'storpool',
'vendor_name': 'StorPool',
'driver_version': self.VERSION,
@@ -1104,5 +1107,5 @@
+ ``iscsi_export_to`` is set to ``*`, that is, all StorPool volumes will be
+ exported via iSCSI to all initiators.
--
-2.43.0
+2.25.1