-
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
What is a Custom Task? #6457
Comments
IMO, it's clever but I found it a bit confusing too |
If apiVersion is only used for custom task, why don't we rename it to something like CustomApiVersion to avoid confusion? |
Note from @Yongxuanzhang: When we enabled custom tasks by default, users who were accidentally setting APIVersion for Tasks and ClusterTasks experienced a regression, which is that Custom tasks would be created instead of TaskRuns (#6682). The fix for these users is to remove APIVersion. I'd like to find some way to make this less confusing; CustomAPIVersion could help with that as Yongxuan suggested, or maybe this could be a reason to explore the discussion in #6186 again? |
Issues go stale after 90d of inactivity. /lifecycle stale Send feedback to tektoncd/plumbing. |
Stale issues rot after 30d of inactivity. /lifecycle rotten Send feedback to tektoncd/plumbing. |
Rotten issues close after 30d of inactivity. /close Send feedback to tektoncd/plumbing. |
@tekton-robot: 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. |
Today, a spec is considered a
Custom Task
whenAPIVersion
andKind
are non-empty.If a user sets
apiVersion
to"tekton.dev/v1beta1"
andkind
to"Task"
, aCustomRun
will be created instead of aTaskRun
.If a user sets
apiVersion
to"tekton.dev/v1beta1"
andkind
to"Pipeline"
, aCustomRun
will be created instead of aPipelineRun
. This behavior has been been useful for Pipelines in Pipelines Custom Task.This behavior was initially surprising to me when refactoring
isCustomTask
in #6447, but it made sense after looking into it more. Was wondering if it may be surprising to users, especially given that @lbernick also raised similar questions during review of #6447. Opened this issue to make this behavior known so that we can discuss whether we're okay with this or we want to make some changes.cc @tektoncd/core-maintainers
The text was updated successfully, but these errors were encountered: