IBM / ubiquity

Ubiquity
Apache License 2.0
90 stars 26 forks source link

Fix/ub 1103 ds8k storwize lun0 rescan #265

Closed yixuan0825 closed 5 years ago

yixuan0825 commented 5 years ago

The fix has passed the CI testing for DS8k. The fix include:

  1. Add a new struct:ScbeVolumeMapInfo, it will include the Host and LunNumber The LunNumber will be get from GetVolMapping.
  2. Add two new function: function isLun0 to judge if the volume is Lun0 RescanSCSILun0 to scan the Lun0 volume.
  3. In the function RescanSCSILun0, I used WriteFile to rescan the volume. In the design, I wanted to use the command echo to rescan the volume . But the command doesn't work here. So I used WriteFile to rescan the volume, it works well.

This change is Reviewable

coveralls commented 5 years ago

Coverage Status

Coverage increased (+1.3%) to 59.09% when pulling 73c226f05967137385f8624fdb41eafc673e96f3 on fix/UB-1103_ds8k_storwize_lun0_rescan into ae6a0a55dd776cfac7fa96c973b9f7691e2c6e2a on dev.

shay-berman commented 5 years ago

feel free to merge but don't forget to merge it as "Squash and merge" (so we will not see all the 33 commits in the dev branch only 1)