Gilad Chaplik has submitted this change and it was merged.
Change subject: core: No validation for disk space (#828334)
......................................................................
core: No validation for disk space (#828334)
https://bugzilla.redhat.com/828334
In AddVmTemplate command there was no validation for remaining disk space,
and the command was sent to the VDSM in spite that.
The problem was that the check was performed on the template instead of the
VM that whom we're going to create template from.
Change-Id: Id8f4c73f7e09d4a6c2ae475f979bdc11116b6b4d
Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmTemplateCommand.java
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/StorageDomainValidator.java
2 files changed, 3 insertions(+), 3 deletions(-)
Approvals:
Michael Kublin: Looks good to me, approved
Gilad Chaplik: Verified
--
To view, visit
http://gerrit.ovirt.org/7367
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id8f4c73f7e09d4a6c2ae475f979bdc11116b6b4d
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Gilad Chaplik <gchaplik(a)redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com>