Nir Soffer has posted comments on this change. Change subject: Added the option to scan alignment directly according to a config value ......................................................................
Patch Set 5: (1 comment) https://gerrit.ovirt.org/#/c/40402/5//COMMIT_MSG Commit Message: Line 4: Commit: Tal Nisan <tni...@redhat.com> Line 5: CommitDate: 2015-12-30 17:03:04 +0200 Line 6: Line 7: Added the option to scan alignment directly according to a config value Line 8: > Currently there is a problem with Libguestfs/Libvirt with alignment scans s I don't see why a customer would ask to disable direct scanning. The only reason to add a configuration is to have an easy way to revert this in the field, in case it cause major trouble, but I don't see a need for this in this case. So lets simplify and enable it without configuration, on the effected platforms. Please also updat the commit message to explain: - which platforms are effected by this bug - the libvirt bug url for this issue - why we need this workaround - why using this config is a workaround and not the correct solution Line 9: Bug-Url: https://bugzilla.redhat.com/1151838 Line 10: Change-Id: I00d8b681e4d2a2f4fb60e673eefdc807d8e1447d -- To view, visit https://gerrit.ovirt.org/40402 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I00d8b681e4d2a2f4fb60e673eefdc807d8e1447d Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tni...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Tal Nisan <tni...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches