Skip to content
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

PipelineRun and TaskRun Status #3792

Closed
jerop opened this issue Feb 25, 2021 · 6 comments
Closed

PipelineRun and TaskRun Status #3792

jerop opened this issue Feb 25, 2021 · 6 comments
Assignees
Labels
area/roadmap Issues that are part of the project (or organization) roadmap (usually an epic) kind/design Categorizes issue or PR as related to design. lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness.

Comments

@jerop
Copy link
Member

jerop commented Feb 25, 2021

Overview

As discussed in API working group on 02/22 (notes | recording), we need to revisit the design of the Status sections of PipelineRuns and TaskRuns

We agreed that we need to rework the Status sections to be at par with the Spec sections of the API as we work towards V1 release

Problems

Problems in the Status sections include:

Please share other issues you may have with Status

References

/cc @tektoncd/core-maintainers

@jerop jerop mentioned this issue Feb 25, 2021
17 tasks
jerop added a commit to jerop/pipeline that referenced this issue Feb 25, 2021
Adding redesign of PipelineRun and TaskRun Status as a priority in the
2021 roadmap as we work towards V1

tektoncd#3792
@vdemeester
Copy link
Member

embedding the TaskRun and Run status in the PipelineRun status

* takes up a lot of space

* concern when supporting [`Pipelines` in `Pipelines`](https://github.com/tektoncd/experimental/pull/705)

* figure out how [`Results`](https://github.com/tektoncd/results) can help

One thing to consider here is that having the spec used in the status allows us to have a finite source of truth of what was used for that run. Without it, we have no way, while looking back at some runs, to know for sure what what the definition of the pipeline or the task executed. In addition, even though we are not taking advantage of this today, this could allow us to reduce the number of "fetching the definiton spec" (from the cluster, from the oci image, …) as the information is already available in the status.

@jerop jerop added the kind/design Categorizes issue or PR as related to design. label Mar 1, 2021
tekton-robot pushed a commit that referenced this issue Mar 4, 2021
Adding redesign of PipelineRun and TaskRun Status as a priority in the
2021 roadmap as we work towards V1

#3792
@jerop jerop added the area/roadmap Issues that are part of the project (or organization) roadmap (usually an epic) label May 18, 2021
@tekton-robot
Copy link
Collaborator

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 tekton-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Oct 15, 2021
@jerop
Copy link
Member Author

jerop commented Oct 15, 2021

/lifecycle frozen

in the 1.0 roadmap - #3548

@tekton-robot tekton-robot added lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. and removed lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. labels Oct 15, 2021
@lbernick lbernick moved this to Todo in Pipelines V1 Jan 11, 2022
@lbernick
Copy link
Member

/assign

@lbernick
Copy link
Member

lbernick commented May 9, 2022

/close

TEP-0100 has now been implemented, and documentation of status is being tracked in #4660.

@tekton-robot
Copy link
Collaborator

@lbernick: Closing this issue.

In response to this:

/close

TEP-0100 has now been implemented, and documentation of status is being tracked in #4660.

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.

Repository owner moved this from In Progress to Done in Tekton Pipelines Roadmap May 9, 2022
Repository owner moved this from In Progress to Done in Pipelines V1 May 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/roadmap Issues that are part of the project (or organization) roadmap (usually an epic) kind/design Categorizes issue or PR as related to design. lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness.
Projects
Status: Done
Status: Done
Development

No branches or pull requests

4 participants