]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Ensure that lun_id is an int
authorJay S. Bryant <jsbryant@us.ibm.com>
Thu, 1 May 2014 22:13:25 +0000 (17:13 -0500)
committerJay S. Bryant <jsbryant@us.ibm.com>
Thu, 1 May 2014 23:22:08 +0000 (18:22 -0500)
commit861e93a671f4ca4a7d8b08b05055bab9b9e141f8
tree2d956983286cd21eb0926d2de7c9d88d201b1269
parent07f231c5eff4efb41ed5984fc04f4f70064f41e6
Ensure that lun_id is an int

The map_vol_to_host function was treating the lun number
as a string rather than an int.  This was causing attempts
to mount storwize_svc 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
mount of storwize_svc volumes to Hyper-V.

Change-Id: I96d179f2a58e5948c8ba6c9ae618daad7b8d4c86
Closes-bug: 1269915
cinder/tests/test_storwize_svc.py
cinder/volume/drivers/ibm/storwize_svc/helpers.py