]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Fix config registration in cinder volume drivers.
authorJohn Griffith <john.griffith@solidfire.com>
Thu, 30 May 2013 22:05:45 +0000 (16:05 -0600)
committerJohn Griffith <john.griffith@solidfire.com>
Wed, 5 Jun 2013 03:51:17 +0000 (21:51 -0600)
commit0b553dcaf8b158872c472fc0dbd9d46d2fc27d25
treee3c3473411f6e69a6456b4a82c0c49d72995283f
parentc2b8f20329cf044261b27e08e3ea0701c816b2b6
Fix config registration in cinder volume drivers.

The config documentation relies on options being registered
on a modules import.  Our need to move the drivers to using
self.configuration for multi-backend support means that options
wouldn't be loaded until object initialization which breaks
documentation.

This patch puts a dummy CONF init/load back in the drivers. While putting
this change together I came across a number of drivers still using FLAGS,
and even worse a number of drivers using a mixture of FLAGS and CONF and
self.configuraiton.  So most of those are cleaned up here as well.

Note there are two drivers that were not updated at all here:
  1. windows.py
  2. zadara.py

The zadara folks have indicated that they're in the process of updating and
releasing a new version of their driver so I left that as is.

The windows driver needs a bit of work to switch over.

Fixes bug: 1179159

Change-Id: I90165299bf080da17741d027e36e361540da0ff8
25 files changed:
cinder/tests/test_coraid.py
cinder/tests/test_glusterfs.py
cinder/tests/test_scality.py
cinder/tests/test_windows.py
cinder/tests/test_xenapi_sm.py
cinder/volume/drivers/coraid.py
cinder/volume/drivers/emc/emc_smis_common.py
cinder/volume/drivers/emc/emc_smis_iscsi.py
cinder/volume/drivers/glusterfs.py
cinder/volume/drivers/huawei/huawei_iscsi.py
cinder/volume/drivers/lvm.py
cinder/volume/drivers/netapp/iscsi.py
cinder/volume/drivers/netapp/nfs.py
cinder/volume/drivers/nexenta/volume.py
cinder/volume/drivers/nfs.py
cinder/volume/drivers/rbd.py
cinder/volume/drivers/san/hp/hp_3par_common.py
cinder/volume/drivers/san/san.py
cinder/volume/drivers/san/solaris.py
cinder/volume/drivers/scality.py
cinder/volume/drivers/sheepdog.py
cinder/volume/drivers/solidfire.py
cinder/volume/drivers/storwize_svc.py
cinder/volume/drivers/xenapi/sm.py
cinder/volume/drivers/xiv.py