-
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
TEP-0090: Matrix - Get Names of Runs
#5037
Conversation
The following is the coverage report on the affected files.
|
6d42ac4
to
3dfa37d
Compare
[TEP-0090: Matrix][tep-0090] proposed executing a `PipelineTask` in parallel `TaskRuns` and `Runs` with substitutions from combinations of `Parameters` in a `Matrix`. In this change, we add a function to get names of `TaskRuns` from Child References. If none are available, it generates new names for `Runs` based on the `PipelineRun` and `PipelineTask` names. [tep-0090]: https://github.com/tektoncd/community/blob/main/teps/0090-matrix.md
3dfa37d
to
a7587cf
Compare
The following is the coverage report on the affected files.
|
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: abayer 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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
Changes
TEP-0090: Matrix proposed executing a
PipelineTask
inparallel
TaskRuns
andRuns
with substitutions from combinationsof
Parameters
in aMatrix
.In this change, we add a function to get names of
TaskRuns
fromChild References. If none are available, it generates new names for
Runs
based on thePipelineRun
andPipelineTask
names.Related PR: #4958
/kind feature
Submitter Checklist
As the author of this PR, please check off the items in this checklist:
Release Notes