Allon Mureinik has submitted this change and it was merged. Change subject: core: QuotaStorageDependent removal in ExtendImageSizeCommand. ......................................................................
core: QuotaStorageDependent removal in ExtendImageSizeCommand. ExtendImageSizeCommand currently implements QuotaStorageDependent which deals with quota consumptions. The command is being called in UpdateVmDiskCommand in case the disk size is extended, but never reaches the actual consume part in the code as it is constructed as an unquota-dependent command. Moreover, UpdateDiskCommand deals with any case of quota consumption (introduced in https://gerrit.ovirt.org/#/c/39330/, BZ: https://bugzilla.redhat.com/1203262 ) So dealing with that is not needed in that command. Change-Id: I940ab5817cb09e4f6275cbda713ecc37a4bd74fc Signed-off-by: Amit Aviram <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ExtendImageSizeCommand.java 1 file changed, 1 insertion(+), 23 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39866 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I940ab5817cb09e4f6275cbda713ecc37a4bd74fc Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Amit Aviram <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Amit Aviram <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Freddy Rolland <[email protected]> Gerrit-Reviewer: Liron Aravot <[email protected]> Gerrit-Reviewer: Maor Lipchuk <[email protected]> Gerrit-Reviewer: Roy Golan <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
