Change in ovirt-engine[master]: core: No validation for disk space (#828334)

gchaplik at redhat.com gchaplik at redhat.com
Tue Aug 21 09:58:35 UTC 2012


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 at 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 at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin at redhat.com>



More information about the Engine-commits mailing list