Skip to content

chore: Add kfp-kubernetes to the core dependencies#75

Open
kramaranya wants to merge 1 commit into
kubeflow:mainfrom
kramaranya:kfp-k8s-dep
Open

chore: Add kfp-kubernetes to the core dependencies#75
kramaranya wants to merge 1 commit into
kubeflow:mainfrom
kramaranya:kfp-k8s-dep

Conversation

@kramaranya
Copy link
Copy Markdown

@kramaranya kramaranya commented Jan 16, 2026

Description of your changes:
Add kfp-kubernetes to the core dependencies, since pipelines using k8s-specific features require kfp-kubernetes to compile.

Checklist:

Pre-Submission Checklist

Additional Checklist Items for New or Updated Components/Pipelines

  • metadata.yaml includes fresh lastVerified timestamp
  • All required files
    are present and complete
  • OWNERS file lists appropriate maintainers
  • README provides clear documentation with usage examples
  • Component follows snake_case naming convention
  • No security vulnerabilities in dependencies
  • Containerfile included if using a custom base image

Signed-off-by: kramaranya <kramaranya15@gmail.com>
Copilot AI review requested due to automatic review settings January 16, 2026 11:10
@google-oss-prow
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign mprahl for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found 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

@google-oss-prow google-oss-prow Bot requested review from HumairAK and mprahl January 16, 2026 11:10
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds kfp-kubernetes as a core dependency to ensure that pipelines using Kubernetes-specific features can compile without requiring manual installation of the package.

Changes:

  • Added kfp-kubernetes>=2.15.2 to the core dependencies list

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Mateusz-Switala pushed a commit to Mateusz-Switala/pipelines-components that referenced this pull request Mar 19, 2026
…ponents-input-params-validation

AutoML components input parameters validation
Mateusz-Switala added a commit to Mateusz-Switala/pipelines-components that referenced this pull request Apr 2, 2026
* tabulat: make flow more flat, reduce components

Signed-off-by: Mateusz Switala <mswitala@redhat.com>
Assisted-by: Claude Code

* improve impelmentation, add tests, fix checks

Signed-off-by: Mateusz Switala <mswitala@redhat.com>
Assisted-by: Claude Code

* improve unit tests

Signed-off-by: Mateusz Switala <mswitala@redhat.com>
Assisted-by: Cursor

* Improve tieme-series leaderboard components

Signed-off-by: Mateusz Switala <mswitala@redhat.com>
Assisted-by: Claude Code

* changes after Claude review

Signed-off-by: Mateusz Switala <mswitala@redhat.com>
Assisted-by: Claude Code

* add model.json with models metadata

Signed-off-by: Mateusz Switala <mswitala@redhat.com>
Assisted-by: Claude Code

* fix predictor path in leaderboard html

Signed-off-by: Mateusz Switala <mswitala@redhat.com>

* fix unit tests

Signed-off-by: Mateusz Switala <mswitala@redhat.com>
Assisted-by: Cursor

* improve model training

Signed-off-by: Mateusz Switala <mswitala@redhat.com>
Assisted-by: Claude Code

* revert changes for pre-commit.yaml

Signed-off-by: Mateusz Switala <mswitala@redhat.com>
Assisted-by: Claude Code

* increase cpu,memory request for tabular; add cpu,memory request for timeseries

Signed-off-by: Mateusz Switala <mswitala@redhat.com>
Assisted-by: Claude Code

---------

Signed-off-by: Mateusz Switala <mswitala@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants