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

Implement preemption for system jobs. #4710

Merged
merged 10 commits into from
Oct 15, 2018

Conversation

preetapan
Copy link
Contributor

Work in progress PR for preemption.

Currently, the approach implemented special cases network resource asks, by preempting per task rather than per task group. This will allow us to preempt allocations on different network devices for the same task group (e.g scenario - task t1 will preempt allocs using eth0, and task t2 will preempt allocs using eth1).

Wrote several unit test cases in preemption_test.go, but will be adding more next week.

This commit implements an allocation selection algorithm for finding
allocations to preempt. It currently special cases network resource asks
from others (cpu/memory/disk/iops).
@preetapan preetapan merged commit 92836a3 into f-preemption-systemjobs Oct 15, 2018
@preetapan
Copy link
Contributor Author

Closing this and the other PR #4732, both are against a feature branch. I will be rebasing against master since the resources structs have changed quite a bit.

@preetapan preetapan deleted the f-preemption-core branch December 11, 2018 23:05
@github-actions
Copy link

I'm going to lock this pull request because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant