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

Kubernetes edits + open questions #1298

Merged
merged 7 commits into from
Sep 12, 2022
Merged

Kubernetes edits + open questions #1298

merged 7 commits into from
Sep 12, 2022

Conversation

CathPag
Copy link
Collaborator

@CathPag CathPag commented Sep 5, 2022

Signed-off-by: Catherine Paganini 74001907+CathPag@users.noreply.github.com

Describe your changes

Made some edits so it's easier to understand but still have open questions. Some sentences don't make 100% sense to me (please view comments below)

Related issue number or link (ex: resolves #issue-number)

NA

Checklist before opening this PR (put x in the checkboxes)

  • [ x] This PR does not contain plagiarism
    • don’t copy other people’s work unless you are quoting and contributing it to them.
  • [ x] I have signed off on all commits
    • signing off (ex: git commit -s) is to affirm that commits comply DCO.

Signed-off-by: Catherine Paganini <74001907+CathPag@users.noreply.github.com>
@netlify
Copy link

netlify bot commented Sep 5, 2022

Deploy Preview for cncfglossary ready!

Name Link
🔨 Latest commit fcfa399
🔍 Latest deploy log https://app.netlify.com/sites/cncfglossary/deploys/631edc494f6a2a00091130e7
😎 Deploy Preview https://deploy-preview-1298--cncfglossary.netlify.app/kubernetes
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Copy link
Collaborator Author

@CathPag CathPag left a comment

Choose a reason for hiding this comment

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

Added my open questions

content/en/kubernetes.md Outdated Show resolved Hide resolved
It automates the lifecycle of containerized applications on modern infrastructures, functioning as a "datacenter operating system" that manages applications across a [distributed system](/distributed-systems/).

Kubernetes schedules [containers](container/) across [nodes](/nodes/) in a [cluster](/cluster/), bundling several infrastructure constructs, sometimes referred to as "primitives."
These primitives consist of app instances, load balancers, persistent storage, etc., and are composed into applications.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The original text says, "composed into an application." What does that mean? Building an app?

Copy link
Collaborator

Choose a reason for hiding this comment

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

@CathPag
I think it means
The primitives (resources) are composed into an environment to run applications.

@CathPag CathPag added the update term Updates to existing terms label Sep 6, 2022
content/en/kubernetes.md Outdated Show resolved Hide resolved
Co-authored-by: Seokho Son <shsongist@gmail.com>
Signed-off-by: Catherine Paganini <74001907+CathPag@users.noreply.github.com>
Copy link
Collaborator

@iamNoah1 iamNoah1 left a comment

Choose a reason for hiding this comment

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

lgtm

content/en/kubernetes.md Outdated Show resolved Hide resolved
content/en/kubernetes.md Outdated Show resolved Hide resolved
content/en/kubernetes.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@iamNoah1 iamNoah1 left a comment

Choose a reason for hiding this comment

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

spellcheck fails for storages

CathPag and others added 3 commits September 10, 2022 04:52
Co-authored-by: Seokho Son <shsongist@gmail.com>
Signed-off-by: Catherine Paganini <74001907+CathPag@users.noreply.github.com>
Co-authored-by: Seokho Son <shsongist@gmail.com>
Signed-off-by: Catherine Paganini <74001907+CathPag@users.noreply.github.com>
Co-authored-by: Seokho Son <shsongist@gmail.com>
Signed-off-by: Catherine Paganini <74001907+CathPag@users.noreply.github.com>
Copy link
Collaborator

@iamNoah1 iamNoah1 left a comment

Choose a reason for hiding this comment

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

lgtm

Signed-off-by: Catherine Paganini <74001907+CathPag@users.noreply.github.com>
Copy link
Collaborator

@seokho-son seokho-son left a comment

Choose a reason for hiding this comment

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

LGTM :)

content/en/kubernetes.md Outdated Show resolved Hide resolved
Co-authored-by: Seokho Son <shsongist@gmail.com>
Signed-off-by: Noah Ispas <13436814+iamNoah1@users.noreply.github.com>
@iamNoah1 iamNoah1 merged commit 2b012fb into main Sep 12, 2022
@seokho-son seokho-son deleted the CathPag-patch-1 branch February 2, 2023 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang/en for English update term Updates to existing terms
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants