Handle v1beta1.taskObject interface and v1beta1.pipelineObject interface for V1 as the storage version

[JeromeJu]

Since the storage version is going to be swapped to v1, and there are no existing v1 task_interface.go and pipeline_interface.go. We need to either remove the interfaces or to separate them from the versioned packages in pkg/apis/pipeline/v1beta1.

• Remove the task_interface.go and pipeline_interface.go

  • This would require lots of refactor for the existing functions eg. https://github.com/tektoncd/pipeline/blob/main/pkg/reconciler/taskrun/resources/taskref.go#L214
  • The logics for nil checking needs to be changed. eg. for taskspec.go

• Move the task_interface.go and pipeline_interface.go out of the versioned package.

Related with v1 release of swapping the storage version, subtask of #5541

[JeromeJu]

Due to the amount of work for removing task_interface.go and pipeline_interface.go entirely from the codebase, I would go for the route that we move them out of the versioned package.

Also trying to figure out if there is an existing way I can take to circumvent duplicating the efforts of handling this now and its later removal.

[JeromeJu]

This comment is to update the progress on this issue:

• Tried swapping all v1beta1.TaskObject to *v1.Task
  https://github.com/tektoncd/pipeline/pull/6444/files/d28850d28a089e8e04eaac885d797d555c832320#r1150819791
• Tried using generic type to include both v1beta1.ClusterTask and v1.Task but that does not seem to fit in the taskref reconciler logics.

[JeromeJu]

Also similar at https://github.com/tektoncd/pipeline/blob/main/pkg/trustedresources/verify.go#L49
cc @Yongxuanzhang could we swap the verification to v1.Task?

[Yongxuanzhang]

https://github.com/tektoncd/pipeline/blob/main/pkg/trustedresources/verify.go#L49

Yes that would be great. I think supporting v1 task and pipeline is what we want

[JeromeJu]

For v1beta1.pipelineObject interface, after searching through the codebase, there doesn't seem to be ClusterPipeline in place. Opening a PR to replace all v1beta1.pipelineObject to v1beta1.pipeline.

[lbernick]

Closed by #6469 and #6471