-
Notifications
You must be signed in to change notification settings - Fork 17
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
update(docs): update readme to explain what it is briefly #4
Conversation
Sorry, opened it because I wanted to check the bot's behavior. Actually, it's WIP yet. /hold |
/kind documentation |
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.
I'm not a native speaker, but I still left some comments for reference.
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.
I reworked the intro using some hints from the well-reviewed dapr docs which are similar. hope it helps.
/label tide/merge-method-squash |
- Language SDKs used to build scheduler plugins, compiled to wasm. | ||
- The scheduler plugin which loads and runs wasm plugins | ||
|
||
## Motivation |
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.
here's a rephrase of your motivation in perhaps more natural English:
Today, you can extend the kube-scheduler, but it is non-trivial. To customize the scheduler means writing Go, and a complicated build process. Once you've built your scheduler, you have deployment and configuration work to have your cluster use it. This isn't one time work, as it needs to be redone on every upgrade of your cluster.
This project lowers that tension by making the default scheduler capable of loading custom plugins, compiled to WebAssembly (wasm). This removes the deployment burdens above. It also allows plugins to be written in languages besides Go.
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.
Great, I fixed based on your suggestion.
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: codefromthecrypt, sanposhiho 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 |
@kerthcet could you check this one again please? |
/lgtm |
What type of PR is this?
/kind documentation
What this PR does / why we need it:
update docs to explain what it is briefly
Which issue(s) this PR fixes:
Fixes #3
Special notes for your reviewer:
Does this PR introduce a user-facing change?
No.