Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix concurrent workers register duplicate task execution for nextstep #787

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Fix concurrent workers register duplicate task execution for nextstep #787

wants to merge 1 commit into from

Commits on Oct 19, 2022

  1. Fix concurrent workers register duplicate task execution for nextstep… 

    … in the joined node

* When one or more steps (e.g., Step1 and Step2) join one Step 3, it may happen occasionally
  that two workers register two duplicate task executions for one next step.
  See bug report in ystia#786
* Fix: Before registering a new task execution, a worker checks if the task execution has already
  been registered for the given step (at _yorc/tasks/<taskID>/.registeredExecutions/<stepName>)
  When deleting the ".runningExecutions" in notifyEnd(), also delete the ".registeredExecution".
* Note: If the step status is DONE, ERROR, or CANCELED, we still register a task execution.
  This is the case where we resume a workflow. The task executions are registered run again.
  But they will be bypass if the given step is DONE.
    @trihoangvo
    trihoangvo committed Oct 19, 2022
    Configuration menu
    Copy the full SHA
    d0ac20f View commit details
    Browse the repository at this point in the history