Omer Frenkel has submitted this change and it was merged.
Change subject: core: Avoid aborting the current TX when destroying stateless VM
......................................................................
core: Avoid aborting the current TX when destroying stateless VM
problem:
when destroying a VM some actions taken in new transaction
which invokes internal commands that can abort it.
solution:
invoke the code with the posiblle side-effects in a new transaction. the outcome of this
flow
is not relevant, we just need to isolate it in a new TX to prevent rollbacking the
current.
backgroud:
When a stateless VM is destroyed we invoke the a method that treats a VM stop
in the event listener. This direct invocation of the event listerner is exclusive to the
Destroy command
for usually, the treatment is invoked when the VM status, which is reported by VDSM, is
changing.
Since the VM has been destoryed already its not reported anymore and the proccessOnVmStop
must be called manually.
Change-Id: I623e118420a748d091699049242a050248ca1ce1
Signed-off-by: Roy Golan <rgolan(a)redhat.com>
---
M
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/DestroyVmVDSCommand.java
1 file changed, 25 insertions(+), 8 deletions(-)
Approvals:
Roy Golan: Verified
Omer Frenkel: Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/4656
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I623e118420a748d091699049242a050248ca1ce1
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>