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
Seldon ksonnet refactor #1752
Seldon ksonnet refactor #1752
Conversation
/assign @jlewi |
/lgtm |
Most recent test flake is "failed to get container status" #1620 |
Looks like the error is
|
The init phase failed to get a supported version
This is fixed by #1747 @cliveseldon could you please rebase off master and rerun the tests? |
/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 seldon ksonnet so almost same as seldon-core repo * update README for ksonnet json construction for seldon * Updated seldon prototypes * reformat jsonnet
* Update seldon ksonnet so almost same as seldon-core repo * update README for ksonnet json construction for seldon * Updated seldon prototypes * reformat jsonnet
… dir. Fixes kubeflow#4530 kubeflow/pipelines (kubeflow#1752) * Updated to make possible to evict pods even with empty dir * fixed missing change * moved annotation * clean up
This PR makes the ksonnet code almost identical to that which is in the core project seldon-core
Also:
Ambassador json is in the code but is defaulted to false as obviously Kubeflow has Ambassador installation separately.
This change is