-
Notifications
You must be signed in to change notification settings - Fork 443
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Test] E2e Tests for Notebook Examples #2417
Comments
/area testing |
@andreyvelich: Please ensure the request meets the requirements listed here. If this request no longer meets these requirements, the label can be removed In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
I am interested in solving this |
Hi I am interested in adding this feature, could you tell me which notebooks should I start with ? @Electronic-Waste @andreyvelich /assign |
Sorry @gjyotin305, this issue has already been taken by @yehudit1987. I'm glad to help you if you pick another issue:) |
Ok |
Hi @yehudit1987, are you still working on this? |
@Electronic-Waste As you can see @saileshd1402 worked on the E2Es for Notebooks as part of this PR: kubeflow/training-operator#2274. |
@andreyvelich I agree with you. I'll look into the CI for Training Operator first. And also, this is a kind reminder that PR #2437 is blocked by this issue. Looking forward to your earliest contributions @yehudit1987. I'm happy to help you with any questions:) |
What you would like to be added?
We plan to add e2e tests for notebooks in CI/CD, run with papermill.
REF: https://github.com/nteract/papermill
The notebook examples in need of e2e tests are listed in the following table:
Under
examples/v1beta1/kubeflow-pipelines
:Under
examples/v1beta1/sdk
:And I would strongly recommend that we start with examples under
sdk
subdirectory since examples underkubeflow-pipelines
need full kubeflow components on test env and are more difficult :)Why is this needed?
This will help us ensure the correctness of our notebook examples for data scientists.
Love this feature?
Give it a 👍 We prioritize the features with most 👍
The text was updated successfully, but these errors were encountered: