-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Run jsonnet tests as part of kubeflow-e2e workflow #443
Run jsonnet tests as part of kubeflow-e2e workflow #443
Conversation
/retest |
Related to #432 * Create test_jsonnet.py - it goes through all the jsonnet tests and executes it * Add test_jsonnet.py to kubeflow-e2e workflow
/cc @lluunn |
/cc @kkasravi |
This is fantastic! Thank you so much for doing this. /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jlewi 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 |
* Update the notebook-controller image. * It looks like the code may not have actually changed since we last built the image. Nonethless, lets update the tag to use the sha as opposed to a tag like v20190911-e8193317-dirty-cd2831 * Update the tests. * Disable the new test for making sure the images are set because it doesn't handle digests. * Related to kubeflow#444
Related to #432
This change is