-
Notifications
You must be signed in to change notification settings - Fork 579
Integration: KAI Scheduler #3886
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
base: master
Are you sure you want to change the base?
Conversation
@EkinKarabulut would you mind fixing the conflict? Thanks. |
…f explanation on docs
05fda72
to
1a63219
Compare
@@ -0,0 +1,38 @@ | |||
apiVersion: scheduling.run.ai/v2 |
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.
It looks like the queue definition using by other yaml files. Could the content of this file put into other yaml files which are using KAI together? It might benefit the end user to apply easily like other yaml files under samples.
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.
I think either way would work as long as we have a clear documentation.
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.
This is awesome. The example works great. Thank you for the contribution!
Just one small comment. Do you mind running the pre-commit (pre-commit run
)?
Why are these changes needed?
This adds the integration to KAI Scheduler.
What has been done?
Checks