Amit Aviram has posted comments on this change. Change subject: core: Quota consumption for Vm's disk updating refactor. ......................................................................
Patch Set 1: -Verified (1 comment) https://gerrit.ovirt.org/#/c/39398/1/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmDiskCommand.java File backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmDiskCommand.java: Line 704: } Line 705: } Line 706: Line 707: // Generate a RELEASE request Line 708: if (doesOldDiskHadQuota && isQuotaChanged) { > doesOldDiskHadQuota is redundant - just let the infra handle it. It seems that the infra handles null quota only in the CONSUME part, This check has to be made. So I'll keep doesOldDiskHadQuota, but will remove doesNewDiskHadQuota. Line 709: list.add(generateQuotaRequestParameters(oldDiskImage, Line 710: QuotaConsumptionParameter.QuotaAction.RELEASE, Line 711: oldDiskImage.getSizeInGigabytes())); Line 712: } -- To view, visit https://gerrit.ovirt.org/39398 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I838973e9f1e71d36193fb2ea832b18b9308c7248 Gerrit-PatchSet: 1 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: Idan Shaby <[email protected]> Gerrit-Reviewer: Liron Aravot <[email protected]> Gerrit-Reviewer: [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
