From 3474ca236de44bb4ccc11d864554f1922f6c64ae Mon Sep 17 00:00:00 2001 From: Patrick East Date: Thu, 10 Sep 2015 13:00:27 -0700 Subject: [PATCH] Swap the decorator order for PureFCDriver methods Right now the debug trace decorator is logging the decorator method from the FCZM helper and not the actual driver methods its supposed to be. Change-Id: If658393e5e9d1b3e068fc7a200998e8033ada48f Closes-Bug: #1494459 --- cinder/volume/drivers/pure.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cinder/volume/drivers/pure.py b/cinder/volume/drivers/pure.py index 902c3a8e8e1..f4029c738cd 100644 --- a/cinder/volume/drivers/pure.py +++ b/cinder/volume/drivers/pure.py @@ -943,8 +943,8 @@ class PureFCDriver(PureBaseVolumeDriver, driver.FibreChannelDriver): ports = self._array.list_ports() return [port["wwn"] for port in ports if port["wwn"]] - @log_debug_trace @fczm_utils.AddFCZone + @log_debug_trace def initialize_connection(self, volume, connector, initiator_data=None): """Allow connection to connector and return connection info.""" @@ -1010,8 +1010,8 @@ class PureFCDriver(PureBaseVolumeDriver, driver.FibreChannelDriver): return init_targ_map - @log_debug_trace @fczm_utils.RemoveFCZone + @log_debug_trace def terminate_connection(self, volume, connector, **kwargs): """Terminate connection.""" no_more_connections = self._disconnect(volume, connector, **kwargs)