Skip to content

Instantly share code, notes, and snippets.

@WaltHP
Created March 17, 2015 18:37
Show Gist options
  • Save WaltHP/6caa895e882691ddfee9 to your computer and use it in GitHub Desktop.
Save WaltHP/6caa895e882691ddfee9 to your computer and use it in GitHub Desktop.
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