Liron Ar has uploaded a new change for review.

Change subject: [wip] core: OvfReader - set IsManaged to true for managed 
devices
......................................................................

[wip] core: OvfReader - set IsManaged to true for managed devices

When reading ovf data, managed devices were added to the managed devices
map - but the isManaged flag wasn't set to true, causing that possibly
wrong data would be persisted/used.

Related to Bug-Url: https://bugzilla.redhat.com/882644

Change-Id: Ifa51f5aa6655b5b9ae6fd1a8c5e0c206fa9630e6
Signed-off-by: Liron Aravot <[email protected]>
---
M 
backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfReader.java
1 file changed, 1 insertion(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/70/13370/1

diff --git 
a/backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfReader.java
 
b/backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfReader.java
index 8d61105..55e98fd 100644
--- 
a/backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfReader.java
+++ 
b/backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfReader.java
@@ -237,6 +237,7 @@
             vmDevice.setIsReadOnly(Boolean.FALSE);
         }
         if (isManaged) {
+            vmDevice.setIsManaged(true);
             vmBase.getManagedDeviceMap().put(vmDevice.getDeviceId(), vmDevice);
         } else {
             vmBase.getUnmanagedDeviceList().add(vmDevice);


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

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

Reply via email to