Change in ovirt-engine[master]: engine: add 'host_device' column to 'vm_device' table
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(a)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(a)redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com>
Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>