Fixes 3par driver methods that were double locking

There were many cases where the 3par drivers(FC and iSCSI) were double
locking, e.g. we were locking in the individual driver and in ‘common’.
This patch set removes all the locks in ‘common’ except the one on
‘create_cloned_volume’.

The methods ’_get_volume_state’ and ‘_copy_volume’ are only called in
‘create_cloned_volume’, so those locks were removed as well. All the other
methods in ‘common’ are only called by the driver which provide the lock,
except ‘create_volume’ which is also called by ‘create_cloned_volume’.

Change-Id: I660fbd04824bd2837e06480c8298ea5a709fc82a
Fixes: bug 1175791
NetApp bug fix for multibackend scenario.

The multibackend scenario using filter scheduler
did not work with NetApp drivers because of use
of FLAGS. Changed to use configuration to support
the multibackend scenario.

bug 1132637

Change-Id: I8ac5bd6d4269f703c3b3b0aa062c92f4412c25af
Fix calling setUp() method of superclass from tearDown method.

In test_netapp.py, the tearDown function was calling setUp of
superclass instead of calling tearDown of superclass. Fixed it
here.

Change-Id: Iab65e60193edcd47d09d7788b0951ef66d816e38
Mark configuration option netapp_password secret

Mark netapp_password configuration option with secret flag.

fixes bug 1151510

Change-Id: I4b04ae610f61c30e8d061b2be710bb48d7afae66
Exit run_tests with the result code of the test runner.

Change-Id: Ie117e258c3ce78080ab55715daa8cf14f9967ce4
Use nose and openstack nose plugin.

It's a little silly since we're working on removing nose for testr,
but it turns out that the old test run wrapper was in use here, which was
causing all sorts of havoc.

Change-Id: Ib6e31547ec4921f91b6253a1798cb04ca6930c87
Merge "Mark configuration option netapp_password secret"
Merge "NetApp bug fix for multibackend scenario."
Pull newly merged Olso update for 'is' operator

A recent update to common/scheduler/filters/extra_specs_ops.py in
Oslo added new 'is' operator to allow Boolean check. The original
commit message was:

"Boolean values for capabilities don't work because extra_specs are
all converted to unicode. The scheduler will then check, for example,
if the boolean 'True' is equal to the unicode string 'True', and will
always return False. This patch allows admins to specify '<is> True'
in extra_specs, which will compare successfully to boolean True."

Notice extra_specs_ops now relies on strutils from Olso, so this
change pull strutils from Oslo as well.

Fix bug: # 1146306

Change-Id: I83adf707c30274a3862aa8034e72c3361fd19952
Merge "Pull newly merged Olso update for 'is' operator"
Fix volume capacity reporting

cinder-volume crashes when the decimal point is not represented by a dot
but by a comma instead. This crash happens for example with the de_DE
locale.

This change replaces any comma appearing in the vgs command output with
a dot.

Test case added.

Fixes: bug #1151684
Change-Id: I831055fb7fd206a8560688ec4c25d766e92ee0f0
Update tox.ini to support RHEL 6.x.

In order to support running unit tests on RHEL 6.x we need to patch
eventlet with contrib/redhat-eventlet.patch. We already
have support for this in the tools/install_venv_common.py but we
need to make a couple changes to allow tox to consume this:

1) Sync in the latest intall_venv_common.py from oslo. This changes
 patch to use the -N option (ignore already applied patches) and
 makes it safe to call the patching function more than once.

2) Add a new patch_tox_venv.py script in tools.

3) Update tox.ini to call patch_tox_venv.py before it runs tests and
 coverage.

NOTE: This will hopefully go away eventually once this patch lands:

 https://bitbucket.org/eventlet/eventlet/issue/89/add-a-timeout-argument-to-subprocesspopen#comment-3342969

Change-Id: I3e2b07c3f718e4aede5c5f231ff0cdb7721ec885
Fix a few bugs for LeftHand Grizzly

This patch fixes a few issues with the LH driver.
1) attach wasn't working because there was no host
created.
2) get_volume_stats didn't exist.
3) fixed the unit tests to work with the new config
access.

Bug 1062508
Bug 1150663

Change-Id: I6f80c3d5c70b64ca6496e50fedcb6b3ca0e2be43
LIO iSCSI initiator ACL auto-config

Currently, IQNs of remote nova compute nodes must be specified in
cinder.conf for them to be added to LIO's ACLs for LUNs.

This change will handle this at volume-attach time instead.

Change-Id: I278ce737042b15bd4d100d331564c1377bac0c55
Removing flags in RBD in favor of configuration

Not needed since we're using configuration for multi-backend support.

Change-Id: I9634fca5d09ca84ef793e330def13176265dac31
Remove/update unused log arguements in manager

Fixes: bug #1130867
Change-Id: I6f9a9c33530e92fa9ce22bd44ef7c40f7e355b04
Update oslo rpc libraries

Update oslo rpc libraries to capture changes, primarly motivated
by secret=True flag on password config options. Skipping broken,
invalid test case while working on correct fix.

Change-Id: Ibb979189b4a6215f307cb49e4a17070ffc7f0f51
Merge "Fix volume capacity reporting"
Merge "Removing flags in RBD in favor of configuration"
Merge "Fix calling setUp() method of superclass from tearDown method."