The test_pipeline.py has been changed to test_pipeline_v2.py#3136
Conversation
Signed-off-by: madmecodes <ayushguptadev1@gmail.com>
|
Let's tes V1 and V2 here, as we do in the regular kfp test :-) |
Signed-off-by: madmecodes <ayushguptadev1@gmail.com>
you meant these tests right? |
|
seems like test_pipeline_v1.py doesnt supports the same command-line parameters as test_pipeline_v2.py |
|
PS: List and deploy test pipeline uses both V2 and V1 API & Fail to tests uses V2 only The V1 script (test_pipeline_v1.py) doesn't have a test_unauthorized_access function like the V2 script has. It only supports running the pipeline.
|
Signed-off-by: madmecodes <ayushguptadev1@gmail.com>
|
Thank you /lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: juliusvonkohout The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
okay, now i will rebase #3135 |
yes and please also reenable the trigger paths for the seweedfs workflow at the top of the file |
The test_pipeline.py has been changed to test_pipeline_v2.py in #3129
✅ Contributor Checklist