Arik Hadas has posted comments on this change. Change subject: pad memory volume only when the storage domain is file based ......................................................................
Patch Set 2: (1 comment) http://gerrit.ovirt.org/#/c/26407/2//COMMIT_MSG Commit Message: Line 8: Line 9: Memory volume should be padded only when the storage domain in which the Line 10: memory volume resides is file-based device. We used to check the type of Line 11: the storage pool, assuming that all the domain within the pool are from Line 12: the same time. This assumption is not true anymore, as we can have > Typo "time" Done Line 13: different types of storage domains in the same storage pool, in case of Line 14: shared storage. So from now on, we'll check the type of the storage Line 15: domain in which the memory volume is in, and only if it is filed-based Line 16: device we'll pad the volume. -- To view, visit http://gerrit.ovirt.org/26407 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I0cebdbba2cc763963b64dcbc142f979802f18c16 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Arik Hadas <aha...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Arik Hadas <aha...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skriva...@redhat.com> Gerrit-Reviewer: Tal Nisan <tni...@redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeen...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches