feat(sdk)!: remove kfp.deprecated code

Signed-off-by: Chen Sun <[email protected]>
kubeflow · Nov 8, 2024 · f7787e9 · f7787e9
1 parent 1708ce7
commit f7787e9
Show file tree

Hide file tree

Showing 343 changed files with 160 additions and 42,987 deletions.
diff --git a/.gitignore b/.gitignore
@@ -56,10 +56,6 @@ bazel-*
 # VSCode
 .vscode
 
-# test yaml
-sdk/python/tests/compiler/pipeline.yaml
-sdk/python/tests/compiler/testdata/testpackage/pipeline.yaml
-
 # Test temporary files
 _artifacts
 

diff --git a/pytest.ini b/pytest.ini
@@ -1,3 +1,3 @@
 [pytest]
-addopts = --ignore=sdk/python/kfp/deprecated --ignore=sdk/python/kfp/tests
+addopts = --ignore=sdk/python/kfp/tests
 testpaths = sdk/python/kfp
diff --git a/samples/core/AutoML tables/AutoML Tables - Retail product stockout prediction.ipynb b/samples/core/AutoML tables/AutoML Tables - Retail product stockout prediction.ipynb
diff --git a/samples/core/condition/condition.py b/samples/core/condition/condition.py
diff --git a/samples/core/condition/condition_test.py b/samples/core/condition/condition_test.py
@@ -15,12 +15,16 @@
 from __future__ import annotations
 
 import unittest
-import kfp.deprecated as kfp
+
+import kfp
+from kfp.samples.test.utils import debug_verify
+from kfp.samples.test.utils import KfpTask
+from kfp.samples.test.utils import run_pipeline_func
+from kfp.samples.test.utils import TestCase
 import kfp_server_api
 from ml_metadata.proto import Execution
-from .condition import condition
+
 from .condition_v2 import condition as condition_v2
-from kfp.samples.test.utils import KfpTask, debug_verify, run_pipeline_func, TestCase
 
 
 def verify_heads(t: unittest.TestCase, run: kfp_server_api.ApiRun,
@@ -43,24 +47,12 @@ def verify_tails(t: unittest.TestCase, run: kfp_server_api.ApiRun,
 run_pipeline_func([
     TestCase(
         pipeline_func=condition_v2,
-        mode=kfp.dsl.PipelineExecutionMode.V2_ENGINE,
         arguments={"force_flip_result": "heads"},
         verify_func=verify_heads,
     ),
     TestCase(
         pipeline_func=condition_v2,
-        mode=kfp.dsl.PipelineExecutionMode.V2_ENGINE,
         arguments={"force_flip_result": "tails"},
         verify_func=verify_tails,
     ),
-    TestCase(
-        pipeline_func=condition,
-        mode=kfp.dsl.PipelineExecutionMode.V1_LEGACY,
-        arguments={"force_flip_result": "heads"},
-    ),
-    TestCase(
-        pipeline_func=condition,
-        mode=kfp.dsl.PipelineExecutionMode.V1_LEGACY,
-        arguments={"force_flip_result": "tails"},
-    ),
 ])