Liron Aravot has submitted this change and it was merged. Change subject: core: CommandBase - persist command before CDA execution. ...................................................................... core: CommandBase - persist command before CDA execution. If an executed child command CDA will fail, the command isn't be persisted so the failure can't be inspected by the parent command. Change-Id: I6addb0786b181a14046d813ddc366b167c5d8e2a Signed-off-by: Liron Aravot <laravot@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java 1 file changed, 6 insertions(+), 4 deletions(-) Approvals: Ravi Nori: Looks good to me, but someone else must approve Moti Asayag: Looks good to me, approved Liron Aravot: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/45151 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6addb0786b181a14046d813ddc366b167c5d8e2a Gerrit-PatchSet: 19 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot@redhat.com> Gerrit-Reviewer: Moti Asayag <masayag@redhat.com> Gerrit-Reviewer: Ravi Nori <rnori@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>