Mike Kolesnik has uploaded a new change for review. Change subject: core: Set parameters task success only for command ......................................................................
core: Set parameters task success only for command Set the task success only for the commands that actually failed, by default tasks are considered to have ended all successfully. The success is an aggregation of the status of the command itself (would change if it also sent a task) and the status of all son commands, where if one failed then as a whole the parent is considered as failed. This will allow to selectively fail the parent command and all son commands that failed, while the son commands that have finished can clean up after themselves, instead of leaving garbage. This change will not harm current behavior, however, since the VmCommand will set the son commands to failed if the parent command has failed either way. Change-Id: I4366b6606ae83064b313aed17b4f65384e22342b Signed-off-by: Mike Kolesnik <mkole...@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/EntityAsyncTask.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VdcActionParametersBase.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/asynctasks/SetTaskGroupStatusVisitor.java 3 files changed, 22 insertions(+), 25 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/27/4127/1 -- To view, visit http://gerrit.ovirt.org/4127 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I4366b6606ae83064b313aed17b4f65384e22342b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Mike Kolesnik <mkole...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
