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(a)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(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Marek Libra <mlibra(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>