openstack/cinder
Code Issues Proposed changes

Merge "HPE 3par: getWsApiVersion login/logout"

Browse Source
This commit is contained in:
Zuul 2024-12-16 20:39:25 +00:00 committed by Gerrit Code Review
commit 01f7c97376
2 changed files with 9 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
cinder/tests/unit/volume/drivers/hpe/test_hpe3par.py
View File

@ -698,7 +698,9 @@ class HPE3PARBaseDriver(test.TestCase):
mock.call.login(HPE3PAR_USER_NAME, HPE3PAR_USER_PASS)]
get_id_login = [
mock.call.login(HPE3PAR_USER_NAME, HPE3PAR_USER_PASS),
mock.call.getWsApiVersion(),
mock.call.logout(),
mock.call.login(HPE3PAR_USER_NAME, HPE3PAR_USER_PASS),
mock.call.getStorageSystemInfo()]

10
cinder/volume/drivers/hpe/hpe_3par_common.py
View File

@ -310,11 +310,12 @@ class HPE3PARCommon(object):
4.0.20 - Use small QoS Latency value. Bug #2018994
4.0.21 - Fix issue seen during retype/migrate. Bug #2026718
4.0.22 - Fixed clone of replicated volume. Bug #2021941
4.0.23 - Fixed login/logout while accessing wsapi. Bug #2068795
"""
VERSION = "4.0.22"
VERSION = "4.0.23"
stats = {}
@ -436,8 +437,9 @@ class HPE3PARCommon(object):
raise exception.InvalidInput(reason=msg)
def client_logout(self):
LOG.debug("Disconnect from 3PAR REST and SSH %s", self.uuid)
self.client.logout()
if self.client is not None:
LOG.debug("Disconnect from 3PAR REST and SSH %s", self.uuid)
self.client.logout()
def _create_replication_client(self, remote_array):
try:
@ -492,6 +494,8 @@ class HPE3PARCommon(object):
if self._replication_enabled:
self.client = None
raise exception.InvalidInput(ex)
finally:
self.client_logout()
if context:
# The context is None except at driver startup.