From: Jenkins <jenkins@review.openstack.org>
Date: Mon, 13 Oct 2014 07:16:29 +0000 (+0000)
Subject: Merge "NetApp fix for default host type in eseries"
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=57e08a4a63609bb679ad88d56969dc07f8d460b1;p=openstack-build%2Fcinder-build.git

Merge "NetApp fix for default host type in eseries"
---

57e08a4a63609bb679ad88d56969dc07f8d460b1
diff --cc cinder/volume/drivers/netapp/eseries/iscsi.py
index 6b339c018,4f76bafac..f626a4fa4
--- a/cinder/volume/drivers/netapp/eseries/iscsi.py
+++ b/cinder/volume/drivers/netapp/eseries/iscsi.py
@@@ -91,15 -109,14 +111,21 @@@ class Driver(driver.ISCSIDriver)
              if not getattr(self.configuration, flag, None):
                  msg = _('%s is not set.') % flag
                  raise exception.InvalidInput(reason=msg)
 +        if not self.configuration.use_multipath_for_image_xfer:
 +            msg = _('Production use of "%(backend)s" backend requires the '
 +                    'Cinder controller to have multipathing properly set up '
 +                    'and the configuration option "%(mpflag)s" to be set to '
 +                    '"True".') % {'backend': self._backend_name,
 +                                  'mpflag': 'use_multipath_for_image_xfer'}
 +            LOG.warning(msg)
  
      def check_for_setup_error(self):
+         self.host_type =\
+             self.HOST_TYPES.get(self.configuration.netapp_eseries_host_type,
+                                 None)
+         if not self.host_type:
+             raise exception.NetAppDriverException(
+                 _('Configured host type is not supported.'))
          self._check_storage_system()
          self._populate_system_objects()