-
Notifications
You must be signed in to change notification settings - Fork 255
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
Incoherencies between tkn pipeline ls
and tkn task ls
#596
Comments
/kind feature |
/assign |
This was the initial decision, to not have latest |
yes but we can show the satus? |
Which status ? We can display the status of the last run of this Task, yeah, but what is the value for this ? What if the task is ran in 50 different pipeline with complete different parameter ? Let's take the Also, if I have 5 pipeline running at the same time, and this task running 3 times in parallel, which one do I display ? what value does it add there too ? It's a bit simple for |
Can see the thought on why this was left out. I was going to make the argument that tasks are namespaced being a factor in why this makes sense (so it would be limited in terms of what it carries out), but I could see a use where multiple pipelines use the same task in a namespace. Not sure where the discussion is with referencing tasks from other namespaces, but that would further complicate this. I think an interesting thought would be whether this applies to ClusterTasks, but It seems like there is some momentum for ClusterPipelines. So while on the surface this looked like a good idea, I am having some second thoughts. |
/close |
@chmouel: Closing this issue. In response to this:
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. |
If we do a
tkn pipeline ls
we have the start, duration and status field.while if we do a
tkn task ls
we only have an age :It could be nice to homogenize this
The text was updated successfully, but these errors were encountered: