Piotr Kliczewski has submitted this change and it was merged. Change subject: core: RunVm - do not pass compensation context ...................................................................... core: RunVm - do not pass compensation context When running stateless vm, the RunVm command first creates a snapshot for the vm. When running the create snapshot command, the command context is passed to the child command - which means that the command compensation context is passed as well. The compensation in use is NoOp compensation context it causes the executed child command to use that compensation context as well, on case of failure the db changes won't be reverted which will leave us with inconsistent db state. Change-Id: I93a6035b3a0691a0b1640444e49ef92b128d1122 Bug-Url: https://bugzilla.redhat.com/1334105 Signed-off-by: Liron Aravot <laravot@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/RunVmCommand.java 2 files changed, 9 insertions(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved Liron Aravot: Verified -- To view, visit https://gerrit.ovirt.org/57848 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I93a6035b3a0691a0b1640444e49ef92b128d1122 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Liron Aravot <laravot@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot@redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski@gmail.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>