Nir Soffer has posted comments on this change.

Change subject: lvm: Exclude faulty devices from lvm long filter
......................................................................


Patch Set 1:

(2 comments)

http://gerrit.ovirt.org/#/c/31875/1//COMMIT_MSG
Commit Message:

Line 11: minutes (stuck in D state). We have seen getDevicesList command stuck
Line 12: for up to 10 minutes because of faulty devices in the long filter.
Line 13: 
Line 14: We used to build the filter from all multipath devices. Now we build 
the
Line 15: filter only from devices which have at least one active paths.
> s/paths/path/
Done
Line 16: 
Line 17:     # multipath -ll
Line 18:     360060160f4a0300038ed7058b5e9e311 dm-0 DGC     ,VRAID
Line 19:     size=15G features='0' hwhandler='1 emc' wp=rw


Line 12: for up to 10 minutes because of faulty devices in the long filter.
Line 13: 
Line 14: We used to build the filter from all multipath devices. Now we build 
the
Line 15: filter only from devices which have at least one active paths.
Line 16: 
> Consider adding a header here, like "For Exmaple:"
Done
Line 17:     # multipath -ll
Line 18:     360060160f4a0300038ed7058b5e9e311 dm-0 DGC     ,VRAID
Line 19:     size=15G features='0' hwhandler='1 emc' wp=rw
Line 20:     |-+- policy='service-time 0' prio=0 status=enabled


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I6d7a973bcefa95813fdc289847760c0955aca30c
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: Saggi Mizrahi <[email protected]>
Gerrit-Reviewer: Yeela Kaplan <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to