Ori Liel has posted comments on this change. Change subject: restapi: Migrates the brick data and mark it for deletion ......................................................................
Patch Set 6: A couple of questions: 1) Why is it necessary to perform this operation on several bricks together (requiring action on collection)? Can't it be done sequentially, similarly, for example, to starting VMs? (If someone want to start several VMs, he would perform a 'start' action for each VM, not one 'start' for all VMs together). 2) Why is it necessary to first mark the bricks for deletion, and then, on 'commit', delete them? Why not do everything in one operation called 'move' (we have 'move' in other places in the API). And where is this 'commit' action - in a separate patch? Thanks, Ori. -- To view, visit http://gerrit.ovirt.org/19400 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iaa88f3402a25316d1717ee914aa77856fbe802db Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shubhendu Tripathi <[email protected]> Gerrit-Reviewer: Kanagaraj M <[email protected]> Gerrit-Reviewer: Michael Pasternak <[email protected]> Gerrit-Reviewer: Ori Liel <[email protected]> Gerrit-Reviewer: Sahina Bose <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
