Created
March 17, 2015 18:37
-
-
Save WaltHP/6caa895e882691ddfee9 to your computer and use it in GitHub Desktop.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
diff --git a/cinder/volume/drivers/san/hp/hp_3par_common.py b/cinder/volume/drivers/san/hp/hp_3par_common.py | |
index 2e562a3..2eeabd1 100644 | |
--- a/cinder/volume/drivers/san/hp/hp_3par_common.py | |
+++ b/cinder/volume/drivers/san/hp/hp_3par_common.py | |
@@ -748,6 +748,7 @@ class HP3PARCommon(object): | |
'free_capacity_gb': 0, | |
'reserved_percentage': 0, | |
'pools': pools} | |
+ LOG.warning(self.stats) | |
def _get_vlun(self, volume_name, hostname, lun_id=None): | |
"""find a VLUN on a 3PAR host.""" | |
@@ -1066,6 +1067,16 @@ class HP3PARCommon(object): | |
if cpg is not default_cpg: | |
# The cpg was specified in a volume type extra spec so it | |
# needs to be validated that it's in the correct domain. | |
+ # log warning here | |
+ LOG.warning(_LW("'hp3par:cpg' is not supported as an extra spec " | |
+ "in a volume type. CPG's are chosen by " | |
+ "the cinder scheduler, as a pool, from the " | |
+ "cinder.conf entry 'hp3par_cpg', which can " | |
+ "be a list of CPGs.")) | |
+ LOG.debug("using pool %(pool)s instead of %(cpg)s" % | |
+ {'pool': pool, 'cpg': cpg}) | |
+ | |
+ cpg = pool | |
self.validate_cpg(cpg) | |
# Also, look to see if the snap_cpg was specified in volume | |
# type extra spec, if not use the extra spec cpg as the | |
@@ -1146,10 +1157,12 @@ class HP3PARCommon(object): | |
return volume_settings | |
def create_volume(self, volume): | |
- LOG.debug('CREATE VOLUME (%(disp_name)s: %(vol_name)s %(id)s)', | |
+ LOG.debug('CREATE VOLUME (%(disp_name)s: %(vol_name)s %(id)s on' | |
+ '%(host)s)', | |
{'disp_name': volume['display_name'], | |
'vol_name': volume['name'], | |
- 'id': self._get_3par_vol_name(volume['id'])}) | |
+ 'id': self._get_3par_vol_name(volume['id']), | |
+ 'host': volume['host']}) | |
try: | |
comments = {'volume_id': volume['id'], | |
'name': volume['name'], |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment