From a1c0fc3b03f0ec72bd51d242d029f71ff298fe82 Mon Sep 17 00:00:00 2001 From: Tom Swanson Date: Thu, 6 Oct 2016 16:12:40 -0500 Subject: [PATCH] Dell SC: Can incorrectly identify if it is failed over Driver was only checking active_backend_id against None when it should have been checking against None or an empty string. Change-Id: Ib62b28742f32f3522a92f2f140e23ba5472eef8d Closes-Bug: #1631149 --- cinder/volume/drivers/dell/dell_storagecenter_common.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/cinder/volume/drivers/dell/dell_storagecenter_common.py b/cinder/volume/drivers/dell/dell_storagecenter_common.py index 4dbec678722..6e6a5fc4e2f 100644 --- a/cinder/volume/drivers/dell/dell_storagecenter_common.py +++ b/cinder/volume/drivers/dell/dell_storagecenter_common.py @@ -87,7 +87,10 @@ class DellCommonDriver(driver.ConsistencyGroupVD, driver.ManageableVD, self.replication_enabled = True if self.backends else False self.is_direct_connect = False self.active_backend_id = kwargs.get('active_backend_id', None) - self.failed_over = (self.active_backend_id is not None) + self.failed_over = True if self.active_backend_id else False + LOG.info(_LI('Loading %(name)s: Failover state is %(state)r'), + {'name': self.backend_name, + 'state': self.failed_over}) self.storage_protocol = 'iSCSI' self.failback_timeout = 30