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

[YUNIKORN-1222] [SI] preempt pods based on the priorities #67

Closed
wants to merge 3 commits into from

Conversation

manirajv06
Copy link
Contributor

What is this PR for?

SI changes for https://issues.apache.org/jira/browse/YUNIKORN-1184

What type of PR is it?

  • - Feature

Todos

  • - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/YUNIKORN-1222

How should this be tested?

Screenshots (if appropriate)

Questions:

  • - The licenses files need update.
  • - There is breaking changes for older versions.
  • - It needs documentation.

@manirajv06 manirajv06 requested a review from wilfred-s May 26, 2022 16:35
@manirajv06 manirajv06 self-assigned this May 26, 2022
@craigcondit
Copy link
Contributor

@manirajv06 please regenerate the SI code (run make and commit lib/go/si/si.pb.go.

@@ -447,6 +447,8 @@ message AllocationAsk {
// Is this a placeholder ask (true) or a real ask (false), defaults to false
// ignored if the taskGroupName is not set
bool placeholder = 10;
// Does this ask originates the application?
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Grammar: should be Is this ask the originator of the application?

scheduler-interface-spec.md Show resolved Hide resolved
wilfred-s
wilfred-s previously approved these changes Jun 2, 2022
Copy link
Contributor

@wilfred-s wilfred-s left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@wilfred-s wilfred-s self-requested a review June 2, 2022 07:42
@wilfred-s wilfred-s dismissed their stale review June 2, 2022 07:43

files missing

Copy link
Contributor

@wilfred-s wilfred-s left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing two files in the commit: si.proto and lib/go/common/constants.go
Re-run make to generate the files

@manirajv06
Copy link
Contributor Author

Missing two files in the commit: si.proto and lib/go/common/constants.go Re-run make to generate the files

Missing two files in the commit: si.proto and lib/go/common/constants.go Re-run make to generate the files

added those two files

Copy link
Contributor

@wilfred-s wilfred-s left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@wilfred-s wilfred-s closed this in 8c6c3fa Jun 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants