-
Notifications
You must be signed in to change notification settings - Fork 946
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
Capacity plugin implement #3283
Conversation
537ba39
to
3ef1904
Compare
@@ -95,6 +95,10 @@ func (ra *Action) Execute(ssn *framework.Session) { | |||
klog.V(3).Infof("Queue <%s> is overused, ignore it.", queue.Name) | |||
continue | |||
} | |||
if !ssn.Preemptive(queue) { |
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.
Actually, I think it should consider the resource demension. Eg, queue is overused on GPU resource, but CPU/MEM is not overused and incomeing job just need cpu/mem, we can still reclaim from other queues to get CPU/MEM back.
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.
Actually, I think it should consider the resource demension. Eg, queue is overused on GPU resource, but CPU/MEM is not overused and incomeing job just need cpu/mem, we can still reclaim from other queues to get CPU/MEM back.
Updated.
queue := obj.(*api.QueueInfo) | ||
attr := cp.queueOpts[queue.UID] | ||
|
||
canReclaim := !attr.deserved.LessEqual(attr.allocated, api.Zero) |
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.
Mybe all dimentions resource is large than deserved can be regarded as not reclaimable.
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.
Yeah,you are right.
3ef1904
to
e29821f
Compare
e29821f
to
c87f66a
Compare
8965847
to
24f226f
Compare
9fdf693
to
db865c2
Compare
I am a litte confused. The If a user wants a configurable So Why adding another field And what's more, the meaning of If it is a field representing a resource BTW, this plugin can not be used with proportion plugin, am I right? |
It's a replace of |
I see it. thanks. |
74c06da
to
e0ff474
Compare
Signed-off-by: Xuzheng Chang <changxuzheng@huawei.com>
Signed-off-by: Xuzheng Chang <changxuzheng@huawei.com>
Signed-off-by: Monokaix <changxuzheng@huawei.com>
Signed-off-by: Xuzheng Chang <changxuzheng@huawei.com>
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
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: william-wang 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 |
Third part of #3242.
This pr should be merged after volcano-sh/apis#121.