-
Notifications
You must be signed in to change notification settings - Fork 14.4k
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
Add blog post: GSoC ‘20 - Building operators for cluster addons #23432
Conversation
Welcome @somtochiama! |
/assign @justinsb @stealthybox |
Deploy preview for kubernetes-io-master-staging ready! Built with commit 7b5bb4d https://deploy-preview-23432--kubernetes-io-master-staging.netlify.app |
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.
Thanks for submitting this Onyekwere! I've added some comments, also it might read easier if you added whitespace after the word where you use parenthesis. For example instead of this(and then text), maybe this (and some text).
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
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.
thanks for the writeup @somtochiama !
i took a quick pass on the doc and found a couple of minor nits :)
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
@neolit123 @castrojo Thanks for the review😊. I have made some changes in line with your comments |
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
/assign @mrbobbytables |
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.
Nice article Somtochi.
I've made a few suggestions inline. The only really important one is the final comment about the link to Slack.
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
@sftim Thanks for the detailed review. I have made corresponding changes. |
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2020-08-25-gsoc‘20 -building-operators-for-cluster-addons.md
Outdated
Show resolved
Hide resolved
/lgtm |
I think we need to update the filename and slug on the top to be a date in the future right? Otherwise it will publish it in the archives? |
@somtochiama ^ Could you do that? @castrojo How much in advance does it need to be? Would tomorrow be good enough? Or two days? |
Either works for me, if possible PM me in slack as well, I don't want to get stuck in a loop where you're waiting for review and then by the time we get to it the time has passed. |
/lgtm |
/approve Thanks for working on this everyone! |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: castrojo 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 |
This blog post gives an overview of the work done by @somtochiama during the Google Summer of Code internship period with Kubernetes