Yaniv Bronhaim has submitted this change and it was merged.

Change subject: multipath: Rescan also FC devices
......................................................................


multipath: Rescan also FC devices

When rescanning devices, we used to update ISCSI devices, and then run
multipath to update device mapping. However, we never rescanned FC
devices.

We used to believe that "multipath -r" was triggering a rescan of FC
devices and solving this issue. This may have been true in older
versions but certainly does not work with current RHEL release, and was
removed in the previous patch.

This patch adds the missing FC rescan using the issue_lip sysfs api,
recommended in the RHEL Storage Administration Guide.

Change-Id: I7699504f9771232ee0b880f9c83a51fd5b90f40e
Bug-Url: https://bugzilla.redhat.com/1123637
Signed-off-by: Nir Soffer <[email protected]>
Reviewed-on: http://gerrit.ovirt.org/27122
Tested-by: Elad Ben Aharon <[email protected]>
Reviewed-by: Allon Mureinik <[email protected]>
Reviewed-by: Xavi Francisco <[email protected]>
Reviewed-by: Federico Simoncelli <[email protected]>
Reviewed-on: http://gerrit.ovirt.org/30985
Reviewed-by: Yaniv Bronhaim <[email protected]>
---
M vdsm/storage/hba.py
M vdsm/storage/multipath.py
M vdsm/supervdsmServer
3 files changed, 33 insertions(+), 1 deletion(-)

Approvals:
  Nir Soffer: Verified
  Yaniv Bronhaim: Looks good to me, approved
  Allon Mureinik: Looks good to me, but someone else must approve



-- 
To view, visit http://gerrit.ovirt.org/30985
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I7699504f9771232ee0b880f9c83a51fd5b90f40e
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.4
Gerrit-Owner: Nir Soffer <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Elad Ben Aharon <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to