Dan Kenigsberg has posted comments on this change.

Change subject: Live Merge: Proactively extend merge target volume
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.ovirt.org/#/c/35573/2//COMMIT_MSG
Commit Message:

Line 14: cause some over-extension of the merge target volume but the upside is
Line 15: that the merge should complete without active monitoring.  Future
Line 16: versions of oVirt will take advantage of an upcoming libvirt 
enhancement
Line 17: to report write watermarks for all volumes which will allow on-demand
Line 18: volume extension.
Please use

 Label: ovirt-3.5-only

and explain why we don't need this in master. (e.g. is it ok with libvirt 
upstream?)
Line 19: 
Line 20: Change-Id: I3d483f9dfe6e19a9f9fec84950bf2ea0c7ed7946
Line 21: Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1155583


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d483f9dfe6e19a9f9fec84950bf2ea0c7ed7946
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Adam Litke <[email protected]>
Gerrit-Reviewer: Adam Litke <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: Tal Nisan <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to