-
Notifications
You must be signed in to change notification settings - Fork 962
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
add drf e2e cases #905
add drf e2e cases #905
Conversation
Travis tests have failedHey @Thor-wl, TravisBuddy Request Identifier: 40cbb8e0-bdd8-11ea-a595-8f8781864445 |
Travis tests have failedHey @Thor-wl, TravisBuddy Request Identifier: 78ad5c60-bde1-11ea-a595-8f8781864445 |
Signed-off-by: Thor-wl <1187526662@qq.com>
Hey @Thor-wl, TravisCI finished with status TravisBuddy Request Identifier: ec8962a0-bf35-11ea-8212-0f9c14daf924 |
job.name = "j2-drf" | ||
job.tasks[0].req = halfCPU | ||
backfillJob := createJob(ctx, job) | ||
err = waitTasksReady(ctx, backfillJob, int(rep)) |
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.
Can you elaborate how drf work in this case?
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.
drf works when a task who requires less resource and to be schedulered will preempt resource of another task who require more resource and is running
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.
add comments
Signed-off-by: Thor <1187526662@qq.com>
test/e2e/drf.go
Outdated
@@ -0,0 +1,53 @@ | |||
/* | |||
Copyright 2019 The Volcano Authors. |
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.
s/2019/2020
Signed-off-by: Thor <1187526662@qq.com>
[APPROVALNOTIFIER] This PR is APPROVED Approval requirements bypassed by manually added approval. This pull-request has been approved by: Thor-wl 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 |
Signed-off-by: Thor 1187526662@qq.com