Vered Volansky has uploaded a new change for review.

Change subject: core: Fix import VM from ovf allocation checks
......................................................................

core: Fix import VM from ovf allocation checks

The scenario is conducted through ImportVmFromConfigurationCommand, and
is triggered either by importing a vm from ovf, or importing a storage
domain. In both cases the disks already exist in the system, so there's
no space allocation, and therefore no need for space allocation checks,
as done in the derived class. These checks always pass on this command.

Change-Id: Ie80cb29e60c2319261574d1647931f8f5f110462
Bug-Url: https://bugzilla.redhat.com/1136719
Signed-off-by: Vered Volansky <[email protected]>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmFromConfigurationCommand.java
1 file changed, 5 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/83/32583/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmFromConfigurationCommand.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmFromConfigurationCommand.java
index 0d7695f..cdf30d0 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmFromConfigurationCommand.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmFromConfigurationCommand.java
@@ -164,4 +164,9 @@
     protected Guid getSourceDomainId(DiskImage image) {
         return image.getStorageIds().get(0);
     }
+
+    @Override
+    protected boolean handleDestStorageDomains() {
+        return true;
+    }
 }


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie80cb29e60c2319261574d1647931f8f5f110462
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Vered Volansky <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to