Alona Kaplan has submitted this change and it was merged.
Change subject: engine: host_device.vm_id should be engine only field
......................................................................
engine: host_device.vm_id should be engine only field
The vm id that is reported by the vdsm is the one that was passed to it by
the engine.
If a device was attached to a vm via a hook, a vm_id won't be reported.
It means, the vdsm.vm_id doesn't have any extra value and can be ignored
by the engine.
Change-Id: I6865c2414db3ff9d92b7c51f4523d85176e5b0c1
Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com>
---
M
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/HostDeviceDaoDbFacadeImpl.java
M
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java
M packaging/dbscripts/host_device_sp.sql
3 files changed, 6 insertions(+), 17 deletions(-)
Approvals:
Alona Kaplan: Verified
Moti Asayag: Looks good to me, approved
--
To view, visit
https://gerrit.ovirt.org/38887
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I6865c2414db3ff9d92b7c51f4523d85176e5b0c1
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alona Kaplan <alkaplan(a)redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server