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

[blog] Adding introduction blog on KubeEdge - 2019-03-12-kubeedge-k8s-based-edge-intro.md #13158

Merged
merged 15 commits into from Mar 19, 2019

Conversation

skdwriting
Copy link
Contributor

Adding a blog post on Introduction to KubeEdge, a Kubernetes based Edge computing platform.
2019-03-12-kubeedge-k8s-based-edge-intro.md and related images.

Please note that KubeEdge team is also collaborating with Kubernetes IOT/EDGE WORKING GROUP. Also KubeEdge is one of the reference architecture discussed in the workgroup.

skdwriting and others added 7 commits March 12, 2019 10:24
Adding KubeEdge High Level Arch img for the blog
Adding KubeEdge Logo img for blog
Updated all the sections for KubeEdge, a k8s based edge computing platform.
(2019-03-12-kubeedge-k8s-based-edge-intro)
@k8s-ci-robot
Copy link
Contributor

Thanks for your pull request. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please follow instructions at https://git.k8s.io/community/CLA.md#the-contributor-license-agreement to sign the CLA.

It may take a couple minutes for the CLA signature to be fully registered; after that, please reply here with a new comment and we'll verify. Thanks.


Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. label Mar 13, 2019
@k8s-ci-robot k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. language/en Issues or PRs related to English language labels Mar 13, 2019
@m1093782566
Copy link

cc @stewart-yu

@netlify
Copy link

netlify bot commented Mar 13, 2019

Deploy preview for kubernetes-io-master-staging ready!

Built with commit 36e1d8a

https://deploy-preview-13158--kubernetes-io-master-staging.netlify.com

Copy link
Contributor

@stewart-yu stewart-yu left a comment

Choose a reason for hiding this comment

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

Thanks for your PR @skdwriting 👍
And need CLA
/assign @kbarnard10 for review either

2. Clone and Build KubeEdge Cloud and Edge
3. Run Cloud
4. Run Edge

Copy link
Contributor

Choose a reason for hiding this comment

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

have format error, may be need blank line

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry , could not make out the issue.

Copy link
Contributor

Choose a reason for hiding this comment

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

image

Copy link
Contributor

Choose a reason for hiding this comment

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

see the preview, please

Copy link
Contributor Author

Choose a reason for hiding this comment

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

kubeedge-blog-01

Somehow i see in the preview correctly.
However, inserted extra lines and ensure again. Kindly check, Thanks a lot for careful review.

@m1093782566
Copy link

@skdwriting Please sign CLA.

@skdwriting
Copy link
Contributor Author

Thanks for your PR @skdwriting
And need CLA
/assign @kbarnard10 for review either

Signed the CLA. Thanks.

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. and removed cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. labels Mar 14, 2019
@m1093782566
Copy link

cc @CindyXing for IOT/Edge wg opinion.

Review of @stewart-yu fixed for KubeEdge/kubeedge link
@stewart-yu
Copy link
Contributor

ping me back after all tech review done
hint: rebase the commit log more gracefully after all review done
/assign

@kbarnard10 kbarnard10 self-assigned this Mar 15, 2019
Copy link

@CindyXing CindyXing left a comment

Choose a reason for hiding this comment

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

Currently there are a few kubernetes based solutions for edge workload including KubeEdge, Rancher K3s and Virtual Kubelet. There are can more. Different from K3s and virtual kubelet, KubeEdge has its control plane at cloud; and each edge node runs as a worker node in the cluster. Here edge is considered as an extension of cloud. The whole thing is a K8s cluster.

The benefits of KubeEdge are to save resource overhead if run cluster at each edge location; and save OPs cost.

We should call these out and clarify the position KubeEdge project holds.

KubeEdge control plane in cloud and extension part made it mode clear.
@skdwriting
Copy link
Contributor Author

@CindyXing, thanks for the comments. It helps.
Actually some of those points, like the extension of cloud, cost, etc were added in intro and also in the second section. However updated to call out clearer in the intro again. Please check.
Once you are fine, I will squash the commits, then the PR can be merged.

Note: somehow, did not see the reply option to your comment, hence added as a new comment

@CindyXing
Copy link

Done technical review. LGTM. Thanks.

@m1093782566
Copy link

@kbarnard10 @stewart-yu

Per @CindyXing, we have done the technical review. Please give a final approval. Thanks!

@kbarnard10
Copy link
Contributor

@m1093782566 Great - thanks! Will get this published first thing tomorrow.

@m1093782566
Copy link

@kbarnard10

Cool! Thanks!

@stewart-yu
Copy link
Contributor

/approve
@kbarnard10 @zacharysarah
any others opinion? is it time to get it merged?

@skdwriting
Copy link
Contributor Author

@stewart-yu @kbarnard10 @zacharysarah @CindyXing , Thanks a lot.

@kbarnard10
Copy link
Contributor

/lgtm
/approve

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Mar 19, 2019
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: kbarnard10, stewart-yu

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 19, 2019
@k8s-ci-robot k8s-ci-robot merged commit 8097ff0 into kubernetes:master Mar 19, 2019
claudiajkang pushed a commit that referenced this pull request Apr 2, 2019
…-based-edge-intro.md (#13158)

* Create 2019-03-12-kubeedge-k8s-based-edge-intro.md

Initial created.

* Create kubeedge-highlevel-arch.png

* Adding KubeEdge High Level Arch img

Adding KubeEdge High Level Arch img for the blog

* Adding KubeEdge Logo img

Adding KubeEdge Logo img for blog

* Updated KubeEdge intro blog

Updated all the sections for KubeEdge, a k8s based edge computing platform.
(2019-03-12-kubeedge-k8s-based-edge-intro)

* Update 2019-03-12-kubeedge-k8s-based-edge-intro.md

Minor format updates

* Update 2019-03-12-kubeedge-k8s-based-edge-intro.md

* [ReviewFix] 2019-03-12-kubeedge-k8s-based-edge-intro.md

Review of @stewart-yu fixed for KubeEdge/kubeedge link

* [ReviewFix] Format 2019-03-12-kubeedge-k8s-based-edge-intro.md

Linefeed update.

* Update 2019-03-12-kubeedge-k8s-based-edge-intro.md

* Update 2019-03-12-kubeedge-k8s-based-edge-intro.md

* [ReviewFix] 2019-03-12-kubeedge-k8s-based-edge-intro.md

KubeEdge control plane in cloud and extension part made it mode clear.

* Update 2019-03-12-kubeedge-k8s-based-edge-intro.md

* Rename 2019-03-12-kubeedge-k8s-based-edge-intro.md to 2019-03-19-kubeedge-k8s-based-edge-intro.md

* Update 2019-03-19-kubeedge-k8s-based-edge-intro.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/en Issues or PRs related to English language lgtm "Looks good to me", indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants