Change in vdsm[ovirt-3.5]: multiapth: Disable hba rescanning by default

ybronhei at redhat.com ybronhei at redhat.com
Tue Oct 21 07:46:29 UTC 2014


Yaniv Bronhaim has submitted this change and it was merged.

Change subject: multiapth: Disable hba rescanning by default
......................................................................


multiapth: Disable hba rescanning by default

Since commit 674337d8f9 we are scanning both iscsi bus and hbas when
issuing multipath.rescan(). We did not find any bad effects when testing
in our QE lab, but we have reports from customers that these rescans have
bad effects on FC connections.

This patch disables hba rescanning by default, so users can use this
feature if works for them, until we investigate this issue and provide
better solution.

This should have been part of the original patch, since we cannot test
all kinds of storage servers in the same setup and load used by all
users. If hba rescanning cannot be used with certain storage server, it
should be possible to disable this feature.

Change-Id: I812308e54ec2a9eab1ca4115e0456b2dd3990cfb
Bug-Url: https://bugzilla.redhat.com/1152587
Signed-off-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/34176
Reviewed-by: Federico Simoncelli <fsimonce at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/34215
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-by: Yaniv Bronhaim <ybronhei at redhat.com>
---
M lib/vdsm/config.py.in
M vdsm/storage/multipath.py
2 files changed, 9 insertions(+), 1 deletion(-)

Approvals:
  Nir Soffer: Verified
  Yaniv Bronhaim: Looks good to me, but someone else must approve
  Federico Simoncelli: Looks good to me, but someone else must approve
  Dan Kenigsberg: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I812308e54ec2a9eab1ca4115e0456b2dd3990cfb
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list