Michael Pasternak has posted comments on this change.

Change subject: restapi:Remove rebalance and replace brick from API
......................................................................


Patch Set 1: I would prefer that you didn't submit this

(1 inline comment)

....................................................
Commit Message
Line 8: 
Line 9: The rebalance volume and replace brick options need
Line 10: to be rewritten as asynchronous gluster tasks.
Line 11: These options are currently not available in the
Line 12: UI. Removing from the API as well.
i'm not sure you're dong the right thing, the fact they're not available in the 
UI does not mean that shouldn't appear in api, are there any technical 
restrictions for that?
Line 13: 
Line 14: Change-Id: I5ce8126324a6e9575d978cbdd136e78a6bd90626
Line 15: Bug-Url: https://bugzilla.redhat.com/957602


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5ce8126324a6e9575d978cbdd136e78a6bd90626
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Sahina Bose <[email protected]>
Gerrit-Reviewer: Kanagaraj M <[email protected]>
Gerrit-Reviewer: Michael Pasternak <[email protected]>
Gerrit-Reviewer: Shubhendu Tripathi <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to