Skip to content

Commit

Permalink
fix: Fixed running multiple workflows with mutex and memo concurrentl…
Browse files Browse the repository at this point in the history
…y is broken (#11883)

Signed-off-by: shmruin <[email protected]>
  • Loading branch information
shmruin authored and terrytangyuan committed Oct 19, 2023
1 parent 396be72 commit 4b6cdae
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions workflow/controller/operator.go
Original file line number Diff line number Diff line change
Expand Up @@ -1938,6 +1938,7 @@ func (woc *wfOperationCtx) executeTemplate(ctx context.Context, nodeName string,
if node != nil {
fulfilledNode := woc.handleNodeFulfilled(nodeName, node, processedTmpl)
if fulfilledNode != nil {
woc.controller.syncManager.Release(woc.wf, node.ID, processedTmpl.Synchronization)
return fulfilledNode, nil
}
// Memoized nodes don't have StartedAt.
Expand Down

0 comments on commit 4b6cdae

Please sign in to comment.