-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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.
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
Update getNextTasks to support Run retries #4647
Conversation
The following is the coverage report on the affected files.
|
Custom Tasks support retries; however, there are some places in the code base that consider retries only for TaskRuns. This commit updates PipelineRunState.getNextTasks to recognize Run retries and adds tests for this method. No functional changes for TaskRuns.
cbd4001
to
46e3f63
Compare
The following is the coverage report on the affected files.
|
/test pull-tekton-pipeline-integration-tests |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: vdemeester The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/lgtm |
/test pull-tekton-pipeline-alpha-integration-tests |
Changes
Custom Tasks support retries (as of #4327); however, there are some places in the code base that consider
retries only for TaskRuns. This commit updates PipelineRunState.getNextTasks to recognize
Run retries and adds tests for this method. No functional changes for TaskRuns. E2E tests for run retries will be added in a subsequent commit.
/kind bug
Submitter Checklist
As the author of this PR, please check off the items in this checklist:
functionality, content, code)
Release Notes