Change in ovirt-engine[master]: core: several multiple RemoveVmCommand get java.lang.Runtime...
Michael Kublin has submitted this change and it was merged.
Change subject: core: several multiple RemoveVmCommand get java.lang.RuntimeException when
restarting engine
......................................................................
core: several multiple RemoveVmCommand get java.lang.RuntimeException when restarting
engine
The bug was caused because of unneeded update at CommandBase of parameters of async
tasks,
during update were passed parameters of parent command,
the update is unneeded because of parameters are passed during creation of task,
because of update was done partitial we got java.lang.RuntimeException.
Solution: to remove unneded update, also we will have additional benefit:
reduced at least two updates query on every task
Change-Id: Ie885b5d069abf0592454f6834466d3fcb5a47c85
Bug-Url: https://bugzilla.redhat.com/861154
Signed-off-by: Michael Kublin <mkublin(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmCommand.java
2 files changed, 6 insertions(+), 8 deletions(-)
Approvals:
Michael Kublin: Verified
Yair Zaslavsky: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/8291
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie885b5d069abf0592454f6834466d3fcb5a47c85
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Michael Kublin <mkublin(a)redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>