Nir Soffer has posted comments on this change. Change subject: multipath: Disable queuing on all multipath devices ......................................................................
Patch Set 1: (2 comments) http://gerrit.ovirt.org/#/c/32277/1/vdsm/storage/vdsm-multipath.rules.in File vdsm/storage/vdsm-multipath.rules.in: Line 6: # (at your option) any later version. See the files README and Line 7: # LICENSE_GPL_v2 which accompany this distribution. Line 8: # Line 9: Line 10: # Vdsm udev rules for multipath devices. > s/Vdsm/VDSM I guess? I don't like VDSM, and vdsm as a name of a component looks wrong. "Vdsm" is the blessed name according to Dan. Line 11: # Line 12: # Ensure that multipath devices use no_path_retry fail, instead of Line 13: # no_path_retry queue, which is hardcoded in multipath configuration for some Line 14: # devices. Line 12: # Ensure that multipath devices use no_path_retry fail, instead of Line 13: # no_path_retry queue, which is hardcoded in multipath configuration for some Line 14: # devices. Line 15: # Line 16: # See https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Linux/7/html-single/DM_Multipath/index.html > Since we haven't certified VDSM with EL7 yet (IIUC), let's link to the RHEL Ok Line 17: # section 5.6. ISSUES WITH QUEUE_IF_NO_PATH FEATUR. Line 18: # Line 19: -- To view, visit http://gerrit.ovirt.org/32277 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6bfe68f4a8d233afe1bb787ce028ffb4c167469b Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[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
