Skip to content

feat(catalog): remove skill catalog endpoints from v1alpha1 API - #3103

Merged
google-oss-prow[bot] merged 2 commits into
kubeflow:mainfrom
pboyd:remove-alpha-skills-endpoints
Aug 20, 2026
Merged

feat(catalog): remove skill catalog endpoints from v1alpha1 API#3103
google-oss-prow[bot] merged 2 commits into
kubeflow:mainfrom
pboyd:remove-alpha-skills-endpoints

Conversation

@pboyd

@pboyd pboyd commented Aug 19, 2026

Copy link
Copy Markdown
Member

Description

Since the skill catalog has never been released as part of the v1alpha1 API, remove it from there so it's only available as part of the new v1 endpoints.

This includes changes from #3105. The history will be cleaner if we merge #3105, and rebase this one, so I marked this one draft. But we can also merge this one and close #3105, if y'all want.

How Has This Been Tested?

Verified that the skill catalog endpoints return 404 on v1alpha1 and work on v1.

Merge criteria:

  • All the commits have been signed-off (To pass the DCO check)
  • The commits have meaningful messages
  • Automated tests are provided as part of the PR for major new functionalities; testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
  • The developer has manually tested the changes and verified that the changes work.
  • Code changes follow the kubeflow contribution guidelines.

Signed-off-by: Paul Boyd <paul@pboyd.io>
@pboyd
pboyd marked this pull request as draft August 20, 2026 13:19
Assisted-by: Claude Code / OpenCode
Signed-off-by: Paul Boyd <paul@pboyd.io>
@pboyd
pboyd force-pushed the remove-alpha-skills-endpoints branch from caff210 to 59e3904 Compare August 20, 2026 14:12
@rareddy

rareddy commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

@pboyd second commit looks right in removing the skill Python. Can you overwrite 2nd commit that may be simpler?

@pboyd

pboyd commented Aug 20, 2026

Copy link
Copy Markdown
Member Author

@pboyd second commit looks right in removing the skill Python. Can you overwrite 2nd commit that may be simpler?

@rareddy, I'm not sure what you mean about overwriting. I submitted the first commit separately as #3105, if we merge that first, then I can rebase this and it will be a single targeted commit.

@rareddy rareddy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/lgtm

@google-oss-prow

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: rareddy

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@pboyd
pboyd marked this pull request as ready for review August 20, 2026 18:36
@google-oss-prow
google-oss-prow Bot requested a review from dbasunag August 20, 2026 18:36
@google-oss-prow
google-oss-prow Bot merged commit d575847 into kubeflow:main Aug 20, 2026
24 of 25 checks passed
@pboyd
pboyd deleted the remove-alpha-skills-endpoints branch August 20, 2026 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants