]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Code Review / openstack-build / cinder-build.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: ddc5856) | patch
Remove need for CONF acces in brick iscsi
authorJohn Griffith <john.griffith@solidfire.com>
Thu, 26 Sep 2013 21:23:01 +0000 (15:23 -0600)
committerJohn Griffith <john.griffith@solidfire.com>
Thu, 3 Oct 2013 03:23:07 +0000 (21:23 -0600)
commit09f2bea49b853c327762d122d6d9e18f62aed31f
treec2076a7545cec67c4597790157e76db8f724bed3tree | snapshot
parentddc585678df8899b42c6d704a848281caa11967dcommit | diff
Remove need for CONF acces in brick iscsi

At some point we'd like brick to be a standalone lib,
and as such we don't want to have a requirement for
CONF files and having duplicate conf entries across
projects.

The better approach would be to let the projects decide
what they want to use, and how they want defaults to be set
and then pass those settings in via __init__ or when calling
the methods that need them.

Partial-Bug: #1230066

Change-Id: Ib108f1abb2948cb896bdad58070daa7a725a205d
cinder/brick/iscsi/iscsi.py diff | blob | history
cinder/tests/test_iscsi.py diff | blob | history
cinder/volume/driver.py diff | blob | history
cinder/volume/drivers/block_device.py diff | blob | history
cinder/volume/drivers/lvm.py diff | blob | history
etc/cinder/cinder.conf.sample diff | blob | history