Change in ovirt-engine[master]: core: dont update devices if vm is not updated

ofrenkel at redhat.com ofrenkel at redhat.com
Thu Jul 9 08:36:01 UTC 2015


Omer Frenkel has submitted this change and it was merged.

Change subject: core: dont update devices if vm is not updated
......................................................................


core: dont update devices if vm is not updated

vms monitoring should skip updating devices of vm that is not marked for
update. if devices-hash changes, the vm should be in the vmDynamicToSave
list, because the analyzer will identify the hash is different.
this also ensures that the VmManager lock is taken for this vm and its
data was not invalidated by other command.

Change-Id: Idc60abd6ba072fe202f1cd50de2879800a4a0ee8
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1230082
Signed-off-by: Omer Frenkel <ofrenkel at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmsMonitoring.java
1 file changed, 10 insertions(+), 4 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Omer Frenkel: Verified
  Arik Hadas: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Idc60abd6ba072fe202f1cd50de2879800a4a0ee8
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list