volcano
volcano copied to clipboard
feat: add queue-level scheduling policy
What type of PR is this?
Add queue-level scheduling policies
What this PR does / why we need it:
This PR introduces queue-level scheduling policy. As outlined in the implementation section of the design document(https://github.com/volcano-sh/volcano/pull/4163), it enables configuring different Actions and Plugins per queue using the SchedulingPolicy. This design also allows for straightforward extension to job-level policies in the future.
Note:
- Before merging this branch, it’s recommended to first merge https://github.com/volcano-sh/apis/pull/176.
- Related test cases will be demonstrated in this document: Test Plan
Which issue(s) this PR fixes:
Fixes #https://github.com/volcano-sh/volcano/issues/3992
Special notes for your reviewer:
@Monokaix @JesseStutler
Does this PR introduce a user-facing change?
/cc
/cc will review this week 😄
Made the following changes based on the weekly meeting discussion:
- Moved the assignment of
jobinfo.schedulingPolicy,taskinfo.schedulingPolicy,queueinfo.schedulingPolicyto the openSession phase. - Updated the logging related to reading the SchedulingPolicy.
Additionally, it was found that using DefaultSchedulingPolicy conflicts with the extension points in ssn, therefore, the global configuration is still kept in ssn.
Great work!
/assign @JesseStutler /assign @Monokaix
Please rebase master and solve code conflict.
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign monokaix
You can assign the PR to them by writing /assign @monokaix in a comment when ready.
The full list of commands accepted by this bot can be found here.
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment
@ElectricFish7: PR needs rebase.
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.