Change in ovirt-engine[master]: core: RunVmValidator storage allocation

amureini at redhat.com amureini at redhat.com
Wed Oct 8 08:44:43 UTC 2014


Allon Mureinik has submitted this change and it was merged.

Change subject: core: RunVmValidator storage allocation
......................................................................


core: RunVmValidator storage allocation

This patch is a part of a series of patches, adding storage allocation
validations to the system when they're missing, and replacing old
verification usage with unified, new, correct and tested verification.
This patch did this for RunVmValidator.
In doing so, space allocation validation for stateless VM became
redundant and was therefore removed.

Change-Id: Ib3f7c7f31d738f84d356d4e89b05a21d4ca7a118
Bug-Url: https://bugzilla.redhat.com/1149809
Signed-off-by: Vered Volansky <vvolansk at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/RunVmValidator.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/validator/RunVmValidatorTest.java
3 files changed, 19 insertions(+), 54 deletions(-)

Approvals:
  Allon Mureinik: Verified; Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/33800
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib3f7c7f31d738f84d356d4e89b05a21d4ca7a118
Gerrit-PatchSet: 7
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Vered Volansky <vvolansk at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list