Arik Hadas has submitted this change and it was merged.
Change subject: core: Clear device addresses on machine type change
......................................................................
core: Clear device addresses on machine type change
When user changes VM machine type, the device addresses are (possibly)
wrong due to topology changes. To avoid issues with VM startup, the
Engine should drop device addresses when machine type is changed.
Dropping VM devices hash in VmDevicesMonitoring forces
VmDevicesMonitoring to update all information about VM's devices on the
next poll.
Change-Id: I14fedf30e64069605e2c2bd849fbba26b992d5dd
Bug-Url:
https://bugzilla.redhat.com/show_bug.cgi?id=1330988
Signed-off-by: Shmuel Melamud <smelamud(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmCommand.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDeviceDao.java
M
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDeviceDaoImpl.java
M packaging/dbscripts/vm_device_sp.sql
4 files changed, 48 insertions(+), 0 deletions(-)
Approvals:
Eli Mesika: Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
Shmuel Leib Melamud: Verified
Arik Hadas: Looks good to me, approved
Martin Polednik: Looks good to me, but someone else must approve
--
To view, visit
https://gerrit.ovirt.org/57537
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I14fedf30e64069605e2c2bd849fbba26b992d5dd
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shmuel Leib Melamud <smelamud(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 Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com>
Gerrit-Reviewer: Shmuel Melamud <smelamud(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>