From: Jean-Baptiste RANSY <jean-baptiste.ransy@alyseo.com>
Date: Thu, 21 Mar 2013 11:08:53 +0000 (+0100)
Subject: CoraidDriver: do not call login from __init__
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=bea1b96ab5f5be05095a6ce362321fb63de2b0e6;p=openstack-build%2Fcinder-build.git

CoraidDriver: do not call login from __init__

We should not call _login from the driver's __init__.
If the ESM is down, the constructor will fail.

- self._login()

Fixes bug 1157500

Change-Id: Ie57fbb6d3704d270a81cd9eb056d7988989da1f8
(cherry picked from commit 28102bc54ac2793ea17f47c81f6812bb84615834)
---

diff --git a/cinder/volume/drivers/coraid.py b/cinder/volume/drivers/coraid.py
index 8471a0b66..9ee55a82b 100644
--- a/cinder/volume/drivers/coraid.py
+++ b/cinder/volume/drivers/coraid.py
@@ -83,7 +83,6 @@ class CoraidRESTClient(object):
         self.urlOpener = urllib2.build_opener(
             urllib2.HTTPCookieProcessor(self.cookiejar))
         LOG.debug(_('Running with CoraidDriver for ESM EtherCLoud'))
-        self._login()
 
     def _login(self):
         """Login and Session Handler."""