Skip to content

Conversation

@Yikun
Copy link
Member

@Yikun Yikun commented Mar 24, 2022

What changes were proposed in this pull request?

This is PR to doc for basic framework capability for Customized Kubernetes Schedulers.

Why are the changes needed?

Guide user how to use spark on kubernetes custom scheduler

Does this PR introduce any user-facing change?

No

How was this patch tested?

CI passed

Copy link
Member

Choose a reason for hiding this comment

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

What does this refer in Volcano scheduler?

Copy link
Member Author

Choose a reason for hiding this comment

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

Set scheduler hints according to configuration

create pg (contains scheduler hints) according to PodGroupTemplate

existing Pod info dynamically.

such as we can set priority according driver.priority, of course, it didn't use it in volcano current implementations

@Yikun
Copy link
Member Author

Yikun commented Mar 29, 2022

@dongjoon-hyun @holdenk Would you mind take a look again on this? the doc for Customized Kubernetes Schedulers for 3.3 branch.

@Yikun
Copy link
Member Author

Yikun commented Mar 29, 2022

All comments addressed, also sync changes on master: #36001

Yikun added 3 commits March 29, 2022 19:55
### What changes were proposed in this pull request?
This is PR to doc for basic framework capability for Customized Kubernetes Schedulers.

### Why are the changes needed?
Guide user how to use spark on kubernetes custom scheduler

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
CI passed

Closes apache#35869 from Yikun/SPARK-38561.

Authored-by: Yikun Jiang <[email protected]>
Signed-off-by: Holden Karau <[email protected]>
@Yikun Yikun force-pushed the SPARK-38561-3.3 branch from 8cb1f24 to 3e0849d Compare March 29, 2022 11:56
Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

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

+1, LGTM.

dongjoon-hyun pushed a commit that referenced this pull request Mar 29, 2022
…dulers`

### What changes were proposed in this pull request?
This is PR to doc for basic framework capability for Customized Kubernetes Schedulers.

### Why are the changes needed?
Guide user how to use spark on kubernetes custom scheduler

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
CI passed

Closes #35955 from Yikun/SPARK-38561-3.3.

Authored-by: Yikun Jiang <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
@Yikun
Copy link
Member Author

Yikun commented Mar 30, 2022

@dongjoon-hyun Thanks!

srowen pushed a commit that referenced this pull request Apr 2, 2022
…nfiguration and fix typos

### What changes were proposed in this pull request?
Add note for specific scheduler configuration and fix typos

### Why are the changes needed?
We addressed some comments on #35955 when backport to 3.3 branch.

### Does this PR introduce _any_ user-facing change?
No, doc only

### How was this patch tested?
![image](https://user-images.githubusercontent.com/1736354/160582401-ced6071e-61f4-42fe-a62a-d32f42e67a53.png)

SKIP_API=1 bundle exec jekyll serve --watch

Closes #36001 from Yikun/SPARK-38561-FOLLOWUP.

Authored-by: Yikun Jiang <[email protected]>
Signed-off-by: Sean Owen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants