Change in ovirt-engine[master]: engine: add 'host_device' column to 'vm_device' table

alkaplan at redhat.com alkaplan at redhat.com
Mon Dec 21 15:00:22 UTC 2015


Alona Kaplan has submitted this change and it was merged.

Change subject: engine: add 'host_device' column to 'vm_device' table
......................................................................


engine: add 'host_device' column to 'vm_device' table

Since the 'hostdev' property was already reported by the vdsm before this patch,
after adding the 'host_device' column to 'vm_device' table, 
already existing vms will continue report the same hash (since in the vdsm side,
no change was done).
It will cause the 'vm-device' table not to be updated with the 'hostdev' value
(until an actual change will be done to the vm).
In order to solve this, the patch nullifies all the hashes of vms with plugged,
passthrough vnic/s.

Change-Id: I0c5c36365f98f3ed4243562b474b846866d8ba62
Signed-off-by: Alona Kaplan <alkaplan at redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmDevice.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDeviceDaoImpl.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmsMonitoring.java
M packaging/dbscripts/create_views.sql
A packaging/dbscripts/upgrade/04_00_0130_add_host_device_column_to_vm_device.sql
M packaging/dbscripts/vm_device_sp.sql
6 files changed, 44 insertions(+), 8 deletions(-)

Approvals:
  Eli Mesika: Looks good to me, but someone else must approve
  Alona Kaplan: Verified; Passed CI tests
  Martin Betak: Looks good to me, but someone else must approve
  Arik Hadas: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I0c5c36365f98f3ed4243562b474b846866d8ba62
Gerrit-PatchSet: 10
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: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Betak <mbetak at redhat.com>
Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list