Omer Frenkel has submitted this change and it was merged. Change subject: core: reduce compensation scope from hibernate command ......................................................................
core: reduce compensation scope from hibernate command This patch reduce the scope in which the compensation that is defined to switch the VM status back to UP is set in the execute stage of the hibernate vm command. The compensation context is cleared when the first task is created (in transaction) - there's no need to keep the compensation anymore since we don't want the status of the VM to switch to UP while tasks that were created as part of the hibernate command exist. Note: this is only part of the solution for the bug mentioned below. Change-Id: Iec46798fc6ea01ef6b0b4701e1af5ba616edf910 Bug-Url: https://bugzilla.redhat.com/949281 Signed-off-by: Arik Hadas <aha...@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/HibernateVmCommand.java 2 files changed, 59 insertions(+), 32 deletions(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14510 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iec46798fc6ea01ef6b0b4701e1af5ba616edf910 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <aha...@redhat.com> Gerrit-Reviewer: Arik Hadas <aha...@redhat.com> Gerrit-Reviewer: Liron Ar <lara...@redhat.com> Gerrit-Reviewer: Michael Kublin <mkub...@redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skriva...@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com> Gerrit-Reviewer: Ravi Nori <rn...@redhat.com> Gerrit-Reviewer: Roy Golan <rgo...@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
