pipeline icon indicating copy to clipboard operation
pipeline copied to clipboard

Bring taskRun.spec.computeResources to beta

Open lbernick opened this issue 3 years ago • 7 comments

This issue tracks beta promotion and feedback on/issues with taskRun.spec.computeResources.

lbernick avatar Sep 13 '22 16:09 lbernick

Issues go stale after 90d of inactivity. Mark the issue as fresh with /remove-lifecycle stale with a justification. Stale issues rot after an additional 30d of inactivity and eventually close. If this issue is safe to close now please do so with /close with a justification. If this issue should be exempted, mark the issue as frozen with /lifecycle frozen with a justification.

/lifecycle stale

Send feedback to tektoncd/plumbing.

tekton-robot avatar Dec 12 '22 17:12 tekton-robot

Stale issues rot after 30d of inactivity. Mark the issue as fresh with /remove-lifecycle rotten with a justification. Rotten issues close after an additional 30d of inactivity. If this issue is safe to close now please do so with /close with a justification. If this issue should be exempted, mark the issue as frozen with /lifecycle frozen with a justification.

/lifecycle rotten

Send feedback to tektoncd/plumbing.

tekton-robot avatar Jan 11 '23 17:01 tekton-robot

I'd love for this to go to beta, or even better stable. It was introduced in v0.39.0 (August 2022) and I can't see any issues that have been raised against it.

Trying to create a re-usable Pipeline using catalog items like buildah is really hard without this if you have a namespace with a ResourceQuota - have to rely on a large LimitRange default, which is wasteful as applies to all Tasks and init containers without resources set.

jimmyjones2 avatar Jan 27 '23 20:01 jimmyjones2

/remove-lifecycle rotten See above

jimmyjones2 avatar Jan 27 '23 20:01 jimmyjones2

@jimmyjones2 I completely agree that being able to specify compute resources from a taskrun rather than a task is necessary for catalog task reusability. I'm curious whether your use cases are better served by taskRun.spec.computeResources or taskRun.spec.stepSpecs[].computeResources (https://github.com/tektoncd/pipeline/issues/5489)?

lbernick avatar Feb 06 '23 17:02 lbernick

@lbernick taskRun.spec.stepSpecs[].computeResources would be less good, as I'd like to create something generic that doesn't need to know the name of the step (ie. it'll create a PipelineRun from any Pipeline with a task called build, which could be the buildah Task or something else)

jimmyjones2 avatar Feb 07 '23 22:02 jimmyjones2

Thanks for the feedback! We are planning to bring at least one of these features to beta but need to think a bit more about which it will be (or both) since I know some users want a bit more granular control that comes with stepSpecs/sidecarSpecs. The way compute resources work in tekton can be really confusing (since k8s assumes containers run in parallel, but we hack them to run sequentially), so I'm hoping to also gather feedback on which of these features results in the fewest "surprises" or is the least confusing.

lbernick avatar Feb 08 '23 14:02 lbernick

Instead of "computeResources" could I suggest "stepResources". For "computeResources" it's not clear to me what "compute" means... verb or noun. Also the computeResources only apply to steps and not sidecars or initcontainers.

TaskRun is of course sensible but... consider also having it in Tasks as that is the unit of sharing and reasonable defaults are very helpful and the TaskRun could then override if needed.

skaegi avatar Apr 13 '23 15:04 skaegi

Hey, is this ready to go to stable yet?

jimmyjones2 avatar Jun 07 '23 20:06 jimmyjones2

sorry @jimmyjones2, we haven't been able to prioritize this yet, but any updates should be reflected on this issue.

lbernick avatar Jun 14 '23 15:06 lbernick

Issues go stale after 90d of inactivity. Mark the issue as fresh with /remove-lifecycle stale with a justification. Stale issues rot after an additional 30d of inactivity and eventually close. If this issue is safe to close now please do so with /close with a justification. If this issue should be exempted, mark the issue as frozen with /lifecycle frozen with a justification.

/lifecycle stale

Send feedback to tektoncd/plumbing.

tekton-robot avatar Sep 12 '23 16:09 tekton-robot

Any progress on this? Would be great to see this moving forward. I fully agree with what @jimmyjones2 commented above.

michaelsauter avatar Oct 02 '23 07:10 michaelsauter

We are also interested for a beta promotion of this feature. Would love to see some updates. cc: @vdemeester

koustavsaha avatar Oct 05 '23 04:10 koustavsaha

We are also interested on this feature to move Beta, do you guys have any timeline yet?

BoseKarthikeyan avatar Oct 06 '23 07:10 BoseKarthikeyan

I am tentatively adding this to the v0.53 LTS milestone 👼🏼

vdemeester avatar Oct 06 '23 08:10 vdemeester

/assign

vdemeester avatar Oct 17 '23 13:10 vdemeester

/assign @khrm

vdemeester avatar Oct 17 '23 13:10 vdemeester

@vdemeester: GitHub didn't allow me to assign the following users: khrm.

Note that only tektoncd members, repo collaborators and people who have commented on this issue/PR can be assigned. Additionally, issues/PRs can only have 10 assignees at the same time. For more information please see the contributor guide

In response to this:

/assign @khrm

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

tekton-robot avatar Oct 17 '23 13:10 tekton-robot

/assign @khrm

khrm avatar Oct 17 '23 13:10 khrm