]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Ensure that lun_id is an int for NetApp Drivers
authorRushil Chugh <rushil@netapp.com>
Tue, 18 Nov 2014 19:37:17 +0000 (14:37 -0500)
committerRushil Chugh <rushil@netapp.com>
Mon, 15 Dec 2014 16:10:23 +0000 (16:10 +0000)
commit2d0204ff3ca3a7d42fac5722d577e18edd34ea83
tree8a13360ce791b03f01a29c70ad859f70e8063e2a
parent5259bd7f6010f17a129e6bbb1d8e486e93b33b03
Ensure that lun_id is an int for NetApp Drivers

Various NetApp drivers were treating the lun_id as a string
rather than an int.  This was causing attempts to mount NetApp
volumes to Hyper-V nodes to fail as they were checking an integer
type against a unicode type which would fail.

This change casts result_lun to an integer after the value has
gone through the ssh injection attack check.  This way Hyper-V
is able to verify if the found LUN is the target LUN, enabling
mounting of NetApp volumes to Hyper-V.

This change also refactors initialize_connection into some helper
methods so as to enable simpler unit testing of the patchset.

Closes-bug: 1372808

Change-Id: I308b3b2dff315ec33451fb45a30ecd53d5d4c353
cinder/tests/volume/drivers/netapp/dataontap/fakes.py
cinder/tests/volume/drivers/netapp/fakes.py
cinder/tests/volume/drivers/netapp/test_utils.py
cinder/volume/drivers/netapp/dataontap/block_base.py
cinder/volume/drivers/netapp/eseries/iscsi.py
cinder/volume/drivers/netapp/utils.py