Omer Frenkel has posted comments on this change. Change subject: core : Fixing ImagesHandler in order to work with multiple storage domains ......................................................................
Patch Set 1: (2 inline comments) .................................................... File backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImagesHandler.java Line 216: returnValue = false; not sure but maybe you can break the loop here (or the list of images is still relevant for the caller, even if the return value is false?) Line 349 im afraid that we got the vm from the db here so we will have its status refreshed -- To view, visit http://gerrit.ovirt.org/2618 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ida0388922913c970f7c348d68bbd6d742af62ed5 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
