Skip to content

Remove KFP support in favor of tekton#1550

Open
swith005 wants to merge 11 commits into
data-prep-kit:devfrom
swith005:kfp_rem
Open

Remove KFP support in favor of tekton#1550
swith005 wants to merge 11 commits into
data-prep-kit:devfrom
swith005:kfp_rem

Conversation

@swith005
Copy link
Copy Markdown
Collaborator

@swith005 swith005 commented Dec 3, 2025

KFP will no longer be supported in the next release; as a result, traces of KFP will be removed from the repo in favor of tekton.

Signed-off-by: shalisha witherspoon <switherspoon005@gmail.com>
Signed-off-by: shalisha witherspoon <switherspoon005@gmail.com>
Signed-off-by: shalisha witherspoon <switherspoon005@gmail.com>
@swith005 swith005 requested review from roytman and touma-I December 3, 2025 20:16
Copy link
Copy Markdown
Collaborator

@roytman roytman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @swith005 , looks like there are more files with KFP, especially github actions.
and transforms/.make.cicd.targets, transforms/.make.workflowsand scripts/k8s-setup

When a new transform is added to the repository,

1. Run `make` in this directory to create the new test .yml for all transforms found in transforms/{universal,code,language} directories
1. Run `make` in this directory to create the new test .yml for all transforms found in transforms/{universal,code,language, images} directories
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think the images dir relates to KFP.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why doesn't images apply? these are for ci/cd tests I believe

@swith005
Copy link
Copy Markdown
Collaborator Author

swith005 commented Dec 4, 2025

Hi @swith005 , looks like there are more files with KFP, especially github actions. and transforms/.make.cicd.targets, transforms/.make.workflowsand scripts/k8s-setup

thanks, let me review further

Signed-off-by: shalisha witherspoon <switherspoon005@gmail.com>
Signed-off-by: shalisha witherspoon <switherspoon005@gmail.com>
@roytman
Copy link
Copy Markdown
Collaborator

roytman commented Dec 11, 2025

the files transforms/.make.workflows and transforms/Makefile.kfp.template should probably be removed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants