Skip to content

Commit

Permalink
fix: Panic in Workflow Retry (#8243)
Browse files Browse the repository at this point in the history
Signed-off-by: Saravanan Balasubramanian <sarabala1979@gmail.com>
  • Loading branch information
sarabala1979 committed Apr 17, 2022
1 parent c95de6b commit ba795e6
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion workflow/sync/sync_manager.go
Original file line number Diff line number Diff line change
Expand Up @@ -192,7 +192,9 @@ func (cm *Manager) Release(wf *wfv1.Workflow, nodeName string, syncRef *wfv1.Syn
syncLockHolder.removeFromQueue(holderKey)
log.Debugf("%s sync lock is released by %s", lockName.EncodeName(), holderKey)
lockKey := lockName.EncodeName()
wf.Status.Synchronization.GetStatus(syncRef.GetType()).LockReleased(holderKey, lockKey)
if wf.Status.Synchronization != nil {
wf.Status.Synchronization.GetStatus(syncRef.GetType()).LockReleased(holderKey, lockKey)
}
}
}

Expand Down

0 comments on commit ba795e6

Please sign in to comment.