Michael Kublin has posted comments on this change. Change subject: engine: quota support for multiple storage images ......................................................................
Patch Set 9: "Hi Michael, please see my comments inline. I think using a list for images will gain us more flexibility advantage. The list should be used also to pass type (raw / preallocated) when creating a VM from template, and can be used for other commands and operations as well" - I don't, there are only one way - or u introducing something and start using it, or not. In your patch I did not see any use of it. Future flexibility it is code which at 99% removed. -- To view, visit http://gerrit.ovirt.org/3090 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3e4b7e28fb9dd3069e86bd8fdf41255398fd1e58 Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Gilad Chaplik <[email protected]> Gerrit-Reviewer: Maor Lipchuk <[email protected]> Gerrit-Reviewer: Michael Kublin <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
