Change in vdsm[master]: hsm : Do not resize on Disconnect Storage Server

frolland at redhat.com frolland at redhat.com
Mon Nov 16 07:57:15 UTC 2015


Freddy Rolland has posted comments on this change.

Change subject: hsm : Do not resize on Disconnect Storage Server
......................................................................


Patch Set 1:

(3 comments)

https://gerrit.ovirt.org/#/c/48578/1//COMMIT_MSG
Commit Message:

Line 8: 
Line 9: When a host is moved to maintenance state, it will
Line 10: disconnect from the connected storage servers.
Line 11: 
Line 12: Then, the sd cache refreshStorage is called. It will rescan
> either "the sd cache's refrshStorage method" or "sdCache.refreshStorage()".
Done
Line 13: the devices, and an attempt to resize them is performed.
Line 14: 
Line 15: However,in case of disconnectStorageServer flow, resizing
Line 16: devices is not needed and may cause exceptions when trying


Line 11: 
Line 12: Then, the sd cache refreshStorage is called. It will rescan
Line 13: the devices, and an attempt to resize them is performed.
Line 14: 
Line 15: However,in case of disconnectStorageServer flow, resizing
> Missing whitespace after the comma.
Done
Line 16: devices is not needed and may cause exceptions when trying
Line 17: to access sysfs when the iscsi connections are being removed.
Line 18: 
Line 19: Change-Id: I78e236a851b7a0e9ab8f3c86e180cc50fde4ad56


Line 13: the devices, and an attempt to resize them is performed.
Line 14: 
Line 15: However,in case of disconnectStorageServer flow, resizing
Line 16: devices is not needed and may cause exceptions when trying
Line 17: to access sysfs when the iscsi connections are being removed.
> ... and needlessly delay the maintenance operation.
Done
Line 18: 
Line 19: Change-Id: I78e236a851b7a0e9ab8f3c86e180cc50fde4ad56
Line 20: Bug-Url: https://bugzilla.redhat.com/1281909


-- 
To view, visit https://gerrit.ovirt.org/48578
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I78e236a851b7a0e9ab8f3c86e180cc50fde4ad56
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Ala Hino <ahino at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Fred Rolland <frolland at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list