Change in ovirt-engine[master]: core: Cluster update triggers VM update only if necessary

[ahadas at redhat.com]

Arik Hadas has submitted this change and it was merged.

Change subject: core: Cluster update triggers VM update only if necessary
......................................................................


core: Cluster update triggers VM update only if necessary

There was a problem when user tried to edit Cluster while there was a
migrating VM in the Cluster. The root cause was collision of VM locks
of MigrateVmCommand and UpdateVmCommand called by UpdateClusterCommand.

This patch doesn't solve the problem entirely. It only lowers the
probability its occurrence by triggering UpdateVmCommand only if cluster
compatibility level was changed.

Change-Id: Ieccac1f08847681d5f326b9c54a99364720cfdb0
Bug-Url: https://bugzilla.redhat.com/1349526
Signed-off-by: Jakub Niedermertl <jniederm at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateClusterCommand.java
1 file changed, 8 insertions(+), 3 deletions(-)

Approvals:
  Jakub Niedermertl: Verified
  Jenkins CI: Passed CI tests
  Marek Libra: Looks good to me, but someone else must approve
  Arik Hadas: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ieccac1f08847681d5f326b9c54a99364720cfdb0
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Jakub Niedermertl <jniederm at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jakub Niedermertl <jniederm at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Marek Libra <mlibra at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>