Tal Nisan has submitted this change and it was merged. Change subject: core: Remove redundant threshold call ......................................................................
core: Remove redundant threshold call While adding storage domains to a map in ImagesHandler.fillImagesMapBasedOnTemplate(), there's a threshold check. This check is redundant and buggy here. Redundant because there are storage allocation checks in the system now, which should be called in due time. Buggy, since in case the threshold has been met, the threshold validation fails, but with no error at the time of failure. The map is not filled correctly, and we get another, unrelated error. This patch removes the threshold validation and relies on it being called when it should. Change-Id: Ifb4357a19bc51ae5feb3c8e79ff36f3adacc327f Bug-Url: https://bugzilla.redhat.com/1185613 Related-To: https://bugzilla.redhat.com/1179690 Signed-off-by: Vered Volansky <vvola...@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommonVmPoolWithVmsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImagesHandler.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MoveOrCopyTemplateCommand.java 4 files changed, 7 insertions(+), 9 deletions(-) Approvals: Allon Mureinik: Looks good to me, approved Vered Volansky: Verified -- To view, visit http://gerrit.ovirt.org/37254 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifb4357a19bc51ae5feb3c8e79ff36f3adacc327f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Vered Volansky <vvola...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Tal Nisan <tni...@redhat.com> Gerrit-Reviewer: Vered Volansky <vvola...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
