-
Notifications
You must be signed in to change notification settings - Fork 321
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
extended job APIs #1537
extended job APIs #1537
Conversation
It requires rebasing |
Signed-off-by: Boris Lublinsky <blublinsky@hotmail.com>
@z103cb, @astefanutti, pls take a look cc: @kevin85421 |
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.
There are two things, that I think should be included in this PR:
- update of the README.md with examples
- the func ValidateCreateJobRequest(request *api.CreateRayJobRequest) error { function should be enhanced to validate the required fields in the ray submitter is present in the request. I would go as far as making the object required if a cluster selector is used.
Implemented check in util/job.go |
Add documentation on how to use it in Readme |
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.
Small nitpick on the port number, it is looking good otherwise.
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
@astefanutti all ready for you |
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.
Left a couple of comments. Will LGTM after that.
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
Why are these changes needed?
In the Job APIs for the apiserver added job submitter parameters. This fixes the problem of Python/Ray version mismatch in the case of existing Ray cluster
Related issue number
Checks