]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Remove global conf settings from iscsi helper
authorJohn Griffith <john.griffith@solidfire.com>
Wed, 11 Jun 2014 22:43:41 +0000 (22:43 +0000)
committerJay S. Bryant <jsbryant@us.ibm.com>
Fri, 20 Jun 2014 17:55:38 +0000 (12:55 -0500)
commit1b21d499bb3bb1e94e2815cbada86db8e8e21406
tree52ab8c03b62acbfd068f5fbcd73d5619bab0610b
parentbd4e556e685a3b7046f418aebe389e066310d7eb
Remove global conf settings from iscsi helper

This "intermediate" iscsi helper pulls all of it's
config settings from the global config.  This is fine
if you only have a single backend, but if you do
multi-backend it puts things in a bad state where some
of the backend specific settings are picked up but
others are not (for example iscsi_ip_address).

This change modifies methods like create_export in
the volume/iscsi helper to take the drivers version
of the config settings as a parameter and use those
instead of setting off of the global values.

Long term there's a lot of cleanup surrounding our
inheritance model and especially the iscsi helpers.
We can address that going forward but here we just want
to fix the bug in the safest way possible.

Change-Id: If17ec3ffb3f4ea7f95da65781885dcd613b1a807
Closes-Bug: 1325799
cinder/volume/drivers/block_device.py
cinder/volume/drivers/lvm.py
cinder/volume/iscsi.py