Change in vdsm[master]: pad memory volume only when the storage domain is file based

ahadas at redhat.com ahadas at redhat.com
Thu Apr 3 21:24:40 UTC 2014


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 <ahadas at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list