kubeflow / pipelines

Machine Learning Pipelines for Kubeflow
https://www.kubeflow.org/docs/components/pipelines/
Apache License 2.0
3.62k stars 1.63k forks source link

WIP: test: Re-enable KFP Dockerfile-CLI component SDK unit test #11305

Open DharmitD opened 1 month ago

DharmitD commented 1 month ago

Removed the skip unit test decorator that was previously added to bypass the test due to failure (referencing issue #11038).

The issue reported in #11038 was caused due to the Dockerfile referencing to Python 3.7. With the Python version updated to 3.9 (done as part of this commit), the test now passes consistently. This commit re-enables the test to ensure proper validation of Dockerfile creation and KFP package.

Description of your changes:

Checklist:

google-oss-prow[bot] commented 1 month ago

[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 connor-mccarthy for approval. For more information see the Kubernetes Code Review Process.

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

Needs approval from an approver in each of these files: - **[sdk/OWNERS](https://github.com/kubeflow/pipelines/blob/master/sdk/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment