Change in ovirt-engine[ovirt-engine-3.6.2]: core: CoCoAsyncTaskHelper - get parent command id from the p...

Tal Nisan has submitted this change and it was merged. Change subject: core: CoCoAsyncTaskHelper - get parent command id from the parent parameters ...................................................................... core: CoCoAsyncTaskHelper - get parent command id from the parent parameters When choosing parent parameters for the tasks the parent command id should be derived from those parameters, otherwise we can get into situation where the parentCommandId will be Guid.Empty not matching the selected parent parameters. This situation leads to the tasks to be considered as of different commands which leads to the endAction() method to be called multiple times when the parent command creates tasks. An example would be the usage of the memory builders, when creating a vm snapshot with memory the endAction() will be executed multiple times, one time for the disks tasks and one time for the memory tasks. Change-Id: I0cacc023e855c8e9651e7134234759288b93e897 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1293663 Signed-off-by: Liron Aravot <laravot@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/CoCoAsyncTaskHelper.java 1 file changed, 5 insertions(+), 4 deletions(-) Approvals: Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved Liron Aravot: Verified -- To view, visit https://gerrit.ovirt.org/51183 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0cacc023e855c8e9651e7134234759288b93e897 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6.2 Gerrit-Owner: Liron Aravot <laravot@redhat.com> Gerrit-Reviewer: Daniel Erez <derez@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot@redhat.com> Gerrit-Reviewer: Moti Asayag <masayag@redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>
participants (1)
-
tnisan@redhat.com