Change in ovirt-engine[ovirt-engine-3.3]: core: backward compatibility for monitor devices

iheim at redhat.com iheim at redhat.com
Wed Feb 5 12:37:36 UTC 2014


Itamar Heim has submitted this change and it was merged.

Change subject: core: backward compatibility for monitor devices
......................................................................


core: backward compatibility for monitor devices

Before v3.1 we used to have just one item for monitors in the OVF file
which contained an element called VirtualQuantity that represented the
number of monitors the VM has.

Now we're setting monitor device per monitor item in the OVF file,
assuming that we have a separate item for each monitor. Since it doesn't
hold for OVF that was created in versions which are earlier that 3.1,
when we import a VM that was exported in version 3.0 and below, it will
always have just one monitor device.

The solution is that in case we import a VM that was exported in version
which is early than 3.1, the number of monitor devices that will be
created will be according to the VirtualQuantity value.

Change-Id: I00a88f7856bc7f356f3ac5fd6eac8e432f659d09
Bug-Url: https://bugzilla.redhat.com/1024636
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfVmReader.java
1 file changed, 10 insertions(+), 1 deletion(-)

Approvals:
  Omer Frenkel: Looks good to me, approved
  Arik Hadas: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I00a88f7856bc7f356f3ac5fd6eac8e432f659d09
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.3
Gerrit-Owner: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Itamar Heim <iheim at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list