Change in ovirt-engine[master]: core: do not treat video device as bootable device

Code Review gerrit at ovirt.org
Sun Jan 29 10:55:27 UTC 2017


>From Arik Hadas <ahadas at redhat.com>:

Arik Hadas has submitted this change and it was merged.

Change subject: core: do not treat video device as bootable device
......................................................................


core: do not treat video device as bootable device

Recent change in VmInfoBuilderImpl added video device to the collection named
managedDevices. This collection is only used to set the boot order on bootable
devices. Since video device is not a bootable device, it should be added to
that collection.

The name of the collection is confusing, thus this patch renames it so it will
better reflect its intended use.

Change-Id: If2108bd0b25015c8bd9a3ba9344dc5954bba90c2
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/builder/vminfo/VmInfoBuilderImpl.java
1 file changed, 11 insertions(+), 12 deletions(-)

Approvals:
  Shahar Havivi: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Arik Hadas: Verified



-- 
To view, visit https://gerrit.ovirt.org/71332
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If2108bd0b25015c8bd9a3ba9344dc5954bba90c2
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: Sharon Gratch <sgratch at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list