Skip to content

Commit

Permalink
Re-enable test_dockerfile_can_contain_custom_kfp_package
Browse files Browse the repository at this point in the history
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, the test now passes consistently.
This commit re-enables the test to ensure proper validation of Dockerfile creation and KFP package.

Signed-off-by: ddalvi <ddalvi@redhat.com>
  • Loading branch information
DharmitD committed Oct 22, 2024
1 parent aec2856 commit c0f0486
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions sdk/python/kfp/cli/component_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -579,9 +579,6 @@ def test_existing_dockerfile_can_be_overwritten(self):
COPY . .
'''))

@unittest.skip(
"Skipping this test as it's failing. Refer to https://github.com/kubeflow/pipelines/issues/11038"
)
def test_dockerfile_can_contain_custom_kfp_package(self):
component = _make_component(
func_name='train', target_image='custom-image')
Expand Down

0 comments on commit c0f0486

Please sign in to comment.