Allon Mureinik has posted comments on this change. Change subject: core: Add MoveImageGroup to MoveImageGroupCommand ......................................................................
Patch Set 4: Code-Review+1 (1 comment) Seems to make sense. Liron? http://gerrit.ovirt.org/#/c/16532/4//COMMIT_MSG Commit Message: Line 6: Line 7: core: Add MoveImageGroup to MoveImageGroupCommand Line 8: Line 9: Decouple CopyImageGroupCommand from MoveImageGroupCommand. Line 10: We should use only MoveImageGroupVDSCommand in MoveImageGroupCommand since copy should not be used as part of it. really long line. please break it up so no line is longer than 70 characters. Line 11: Line 12: Change-Id: Ibf34e2b6c3c33dd86fe4b4243c86b01cd1b19f7c -- To view, visit http://gerrit.ovirt.org/16532 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ibf34e2b6c3c33dd86fe4b4243c86b01cd1b19f7c Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Itamar Heim <[email protected]> Gerrit-Reviewer: Liron Ar <[email protected]> Gerrit-Reviewer: Maor Lipchuk <[email protected]> Gerrit-Reviewer: Sergey Gotliv <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
