Skip to content
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

MaxResource of Queue #117

Closed
3 tasks
k82cn opened this issue Apr 30, 2019 · 10 comments · Fixed by volcano-retired/scheduler#7
Closed
3 tasks

MaxResource of Queue #117

k82cn opened this issue Apr 30, 2019 · 10 comments · Fixed by volcano-retired/scheduler#7
Assignees
Labels
kind/feature Categorizes issue or PR as related to a new feature. priority/high
Milestone

Comments

@k82cn
Copy link
Member

k82cn commented Apr 30, 2019

Is this a BUG REPORT or FEATURE REQUEST?:

/kind feature

Description:

Support MaxResource of queue by admission controller & queue controller.

xref kubernetes-retired/kube-batch#452

@k82cn
Copy link
Member Author

k82cn commented May 3, 2019

/kind feature
/priority high

@volcano-sh-bot volcano-sh-bot added kind/feature Categorizes issue or PR as related to a new feature. priority/high labels May 3, 2019
@k82cn k82cn added this to the v0.1 milestone May 4, 2019
@k82cn
Copy link
Member Author

k82cn commented May 6, 2019

/assign @hzxuzhonghu

@k82cn
Copy link
Member Author

k82cn commented May 6, 2019

I'm thinking whether we should enhance enqueue action to consider queue's capability, so we can enqueue "high priority" jobs. And leverage #135 to make sure the pod will not be create if not enough resources.

@k82cn
Copy link
Member Author

k82cn commented May 6, 2019

/cc @wangyuqing4 @jiaxuanzhou

@jiaxuanzhou
Copy link
Contributor

I'm thinking whether we should enhance enqueue action to consider queue's capability, so we can enqueue "high priority" jobs. And leverage #135 to make sure the pod will not be create if not enough resources.

yeah, agree with u, but the behaviors of the queue controller better be designed to support the dynamic changes of the specs(either the pg number or the resource quota), so the update func of queue controller and admission controller could work together to notify the changes to make the decision whether to create or delay the creation of the pods.

@hzxuzhonghu
Copy link
Collaborator

I am not sure what does spec.Capability and MaxResource mean? And also how does it cooperate with spec.Weight. I have no context at all.

@jiaxuanzhou
Copy link
Contributor

@hzxuzhonghu
1, the weight means the weight of resource of the whole cluster for one queue
2, MaxResource is manual defined by the end-user for the queue may be less or more than the weight of the queue

@hzxuzhonghu
Copy link
Collaborator

Then these two fields are both optional, and if both specified, how to deal with the conflict?

@jiaxuanzhou
Copy link
Contributor

Then these two fields are both optional, and if both specified, how to deal with the conflict?

maxResource often used for scenario of resource overmatching, but the weight is for cluster scope, so by my side, weight has higher privilege.

@hzxuzhonghu
Copy link
Collaborator

So I would expect a detailed design doc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature. priority/high
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants