-
Notifications
You must be signed in to change notification settings - Fork 831
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 model artifact versions #4774
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
samples/k8s-multi-version.md
Outdated
``` | ||
|
||
```bash | ||
kubectl create -f ./models/multi-version-1.yaml -n ${NAMESPACE} |
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.
💭 Probably worth using apply
here as we expect to update this later; I realise this is generated from the notebook
@@ -54,6 +55,7 @@ local-examples.md | |||
k8s-examples.md | |||
huggingface.md | |||
model-zoo.md | |||
multi-version.md |
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.
❓ Has this file been added?
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.
will add!
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.
One important question and a minor suggestion, but generally looks good!
I've also checked that the other parts of the model spec are provided in the adaptor between k8s resource and scheduler spec.
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.
Checked this renders fine in docs - looks good!
Fix artifact version is not passed to scheduler from Model yaml.
Fixes: #4770