]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
EQLX: Consolidate CHAP config options
authorRushi Agrawal <rushi.agr@gmail.com>
Sun, 18 Jan 2015 06:53:32 +0000 (12:23 +0530)
committerRushi Agrawal <rushi.agr@gmail.com>
Tue, 17 Feb 2015 05:25:25 +0000 (10:55 +0530)
commit3ffb34e8e92d0aa7e23a470695627434455c9858
treef392a8dfac3259ecbebc4a2f9b83f0b720787a75
parent7cd5fe6bb5b138a3df6665ae7c1fefa4da403f38
EQLX: Consolidate CHAP config options

In order to reduce configuration option bloat, it would be a good exercise
to consolidate configuration options which many drivers implement under
their own distinct name but have same functionality. Here, a first attempt
is made to consolidate CHAP configuration options. Dell's EQLSanISCSIDriver
is made to use these new set of configuration options 'use_chap_auth',
'chap_username' and 'chap_password' instead of the driver-specific options
'eqlx_use_chap', 'eqlx_chap_login' and 'eqlx_chap_password' respectively in
this patch. More drivers are to be added in the future.

DocImpact: Config options of Dell EQLSanISCSIDriver are changed

Change-Id: I5c51301653ad0f76ce0550e12177ad08f20290e0
Partial-Bug: 1398976
cinder/tests/test_eqlx.py
cinder/volume/driver.py
cinder/volume/drivers/eqlx.py