-
Notifications
You must be signed in to change notification settings - Fork 71
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
Fix and improve CI unit test workaround #271
Conversation
Failing CI is a good sign :) Also, having content of the error file right in the run log may be nice thing, since I think that the necessary file isn't saved among the artifacts at the moment, right? |
3e05e54
to
435e3f7
Compare
Yes, only for the Trusty notebook. 🥳
Fixed it! |
1edc072
to
5ea9e76
Compare
…book is broken or doesn't fulfill the unitests
/test notebooks-e2e-tests |
/retest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you! 🥳
/test notebooks-e2e-tests |
Adding here the notebook-e2e-tests job before the ocp-ci went down. /override ci/prow/notebooks-e2e-tests Thank you for reviewing this! 🙂 |
@atheo89: Overrode contexts on behalf of atheo89: ci/prow/notebooks-e2e-tests 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. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: atheo89, jstourac The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
…st-updater-9520338673 [Digest Updater Action] Update Runtime Images
Related to: #269
Improvement of: #176
Fix and improve CI unit test recipe. This can be a temporary solution till figure out the
Possible other improvements
that the the issue is mentioning.Description
This PR improves the code quality of the recipe by utilizing functions and avoiding repetitive code, moreover, when a test_notebook.ipynb is broken or doesn't fulfill the unit tests it returns 0 code and stops the execution of the e2e test with an ERROR message.
How Has This Been Tested?
Below are the logs on how the test behaves in healthy and problematic notebooks:
Problematic notebook:
make test-jupyter-trustyai-ubi9-python-3.9
Healthy Notebook:
make test-jupyter-datascience-ubi9-python-3.9
Merge criteria: