]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
CoraidDriver: typo in _login exception handler
authorJean-Baptiste RANSY <jean-baptiste.ransy@alyseo.com>
Thu, 21 Mar 2013 10:51:37 +0000 (11:51 +0100)
committerJean-Baptiste RANSY <jean-baptiste.ransy@alyseo.com>
Thu, 21 Mar 2013 10:59:40 +0000 (11:59 +0100)
In cinder/volume/drivers/coraid.py, _login method, if login is
unsuccessful, we try to get the message from an undefined
object "response". Instead we should be getting it from "reply".

- errmsg = response.get('message', '')
+ errmsg = reply.get('message', '')

Fixes bug 1157499

Change-Id: Ifd047ced6b40f49881e81d4e73189d9bd216c903

cinder/volume/drivers/coraid.py

index 260a48aeae70813d1de01207b689ddf5622245f5..549a2ea3ca24ecb1ffda30e2769dec7816058acc 100644 (file)
@@ -98,7 +98,7 @@ class CoraidRESTClient(object):
                 LOG.debug(msg % dict(session=self.session))
                 return True
             else:
-                errmsg = response.get('message', '')
+                errmsg = reply.get('message', '')
                 msg = _('Message : %(message)s')
                 raise CoraidESMException(msg % dict(message=errmsg))
         return True