-
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-0118: Apply Param and Result Replacements in Matrix #6345
Conversation
The following is the coverage report on the affected files.
|
The following is the coverage report on the affected files.
|
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.
Thanks @lbernick
515dba1
to
b018457
Compare
The following is the coverage report on the affected files.
|
The following is the coverage report on the affected files.
|
b018457
to
335a5bc
Compare
The following is the coverage report on the affected files.
|
The following is the coverage report on the affected files.
|
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.
@EmmaMunley requesting some changes in the replacements
335a5bc
to
10f0959
Compare
The following is the coverage report on the affected files.
|
The following is the coverage report on the affected files.
|
The following is the coverage report on the affected files.
|
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.
we do not yet support array replacements from array results - #5925
e2d2649
to
238a961
Compare
The following is the coverage report on the affected files.
|
The following is the coverage report on the affected files.
|
The following is the coverage report on the affected files.
|
/test pull-tekton-pipeline-build-tests |
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.
Thank you @EmmaMunley 🎉
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jerop 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 |
This commit updates replacement functionality for applying parameters specific to matrix parameters including matrix include parameters. Matrix.params can have array replacements from array params or string replacements from string, array and object params. Matrix.Include params can only have string replacements from string, array and object params. Note: This feature is still in preview mode.
238a961
to
a2baf31
Compare
The following is the coverage report on the affected files.
|
The following is the coverage report on the affected files.
|
/unhold |
/lgtm |
Please add release note for feature PRs 🙏 |
Changes
[TEP-0090: Matrix] introduced
Matrix
to thePipelineTask
specification such that thePipelineTask
executes a list ofTaskRuns
orRuns
in parallel with the specified list of inputs for aParameter
or with different combinations of the inputs for a set ofParameters
.To build on this, Tep-0018 introduced Matrix.Include, which allows passing in a specific combinations of
Parameters
into theMatrix
.This PR updates replacement functionality for applying parameters specific to matrix parameters including matrix include parameters.
Matrix.params can have array replacements from array params or string replacements from string, array and object params.
Matrix.Include params can only have string replacements from string, array and object params.
/kind feature
Submitter Checklist
As the author of this PR, please check off the items in this checklist:
functionality, content, code)
/kind <type>
. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tepRelease Notes