Change in ovirt-engine[master]: core: Clear device addresses on machine type change

[ahadas at redhat.com]

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 at 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 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 Polednik <mpolednik at redhat.com>
Gerrit-Reviewer: Shmuel Leib Melamud <smelamud at redhat.com>
Gerrit-Reviewer: Shmuel Melamud <smelamud at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>