Amit Aviram has posted comments on this change. Change subject: core: Quota consumption for Vm's disk updating refactor. ......................................................................
Patch Set 1: (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) { > Why not just fix the infra? I'm not sure how deep we need to go inside this issue, and also changing the infra could affect some flows that we are not considering from our point of view 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
