-
Notifications
You must be signed in to change notification settings - Fork 198
docs: clarify differences between binary installation and keadm installation #784
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
base: master
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -10,6 +10,18 @@ In this quick-start guide, we will explain: | |||||
| - A few common ways of deploying KubeEdge. | ||||||
| - Links for further reading. | ||||||
|
|
||||||
| ## Choose an installation method | ||||||
|
|
||||||
| KubeEdge provides two common installation approaches: using `keadm` and installing from binaries manually. | ||||||
|
|
||||||
| - **`keadm`** is recommended for most users who want a simpler and more automated installation experience. | ||||||
| - **Binary installation** is more suitable for users who need more flexibility and full control over binary placement, configuration files, and service setup. | ||||||
|
|
||||||
| | Method | Best for | Automation | Configuration and service setup | | ||||||
| | --- | --- | --- | --- | | ||||||
| | `keadm` | quick start, evaluation, and standard deployments | higher | mostly handled by `keadm` | | ||||||
| | binary installation | custom environments, development, and debugging | lower | managed manually | | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. For consistency with the bullet points above (line 18) and the row above in the table, "Binary installation" should be capitalized.
Suggested change
Comment on lines
+13
to
+23
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. for me, this is kind of duplication though. probably if we want to keep the notification like this, each page should not have the same note at all? |
||||||
|
|
||||||
| ## Dependencies | ||||||
|
|
||||||
| For cloud side, we need: | ||||||
|
|
||||||
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.
The sentence is split across two lines, which introduces an unnecessary line break and a trailing space on line 6. It is recommended to keep this sentence on a single line for better readability and consistency with the rest of the 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 second this, maybe this is just mistyping.