Change in ovirt-engine[master]: engine: host_device.vm_id should be engine only field

alkaplan at redhat.com alkaplan at redhat.com
Mon Mar 23 09:42:31 UTC 2015


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 at 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 at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list