Shahar Havivi has uploaded a new change for review.

Change subject: restapi: set domain for vm-init
......................................................................

restapi: set domain for vm-init

Wrong check for setting domain for VM

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1069201
Change-Id: I9498970ab0135c8978e016cece46e2d3d90b183d
Signed-off-by: Shahar Havivi <shah...@redhat.com>
---
M 
backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/41/25341/1

diff --git 
a/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java
 
b/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java
index bd95d72..4a31a4b 100644
--- 
a/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java
+++ 
b/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java
@@ -295,7 +295,7 @@
             if (staticVm.getVmInit() == null) {
                 staticVm.setVmInit(new VmInit());
             }
-            if (staticVm.getVmInit().getDomain() == null) {
+            if (StringUtils.isEmpty(staticVm.getVmInit().getDomain())) {
                 staticVm.getVmInit().setDomain(vm.getDomain().getName());
             }
         }


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9498970ab0135c8978e016cece46e2d3d90b183d
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shahar Havivi <shav...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to