-
Notifications
You must be signed in to change notification settings - Fork 220
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-0104]: Descope to runtime only #722
Conversation
cec3244
to
cc65577
Compare
/assign @jerop @afrittoli |
@@ -125,17 +122,17 @@ spec: | |||
|
|||
This field should also be added to [PipelineRun.TaskRunSpecs][taskRunSpecs]. |
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.
think we still need this field under PipelineRun
right?
there is no "legacy" Resources
field, but will keep using ComputeResources
for a consistent naming
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.
yes, it still makes sense to have this in PipelineRun.Spec.TaskRunSpecs under "computeResources"
This commit updates TEP-0104 to specify that we will implement Task-level resource requirements for TaskRuns only to start, and we may consider adding support on Tasks in the future.
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 Lee!
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: afrittoli, 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 |
/lgtm approved by both assignees - @afrittoli and @jerop - so merging offline |
This commit updates TEP-0104 to specify that we will implement
Task-level resource requirements for TaskRuns only to start,
and we may consider adding support on Tasks in the future.
/kind tep