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

Add blog post about Storage in Kubernetes #43863

Closed
wants to merge 11 commits into from
Closed

Add blog post about Storage in Kubernetes #43863

wants to merge 11 commits into from

Conversation

GoutamVerma
Copy link

@GoutamVerma GoutamVerma commented Nov 9, 2023

Hello!

I've added the blog on topic "Storage in K8s". It is informative blog that include the complete detail about the storage and storage class in k8s.

Copy link

linux-foundation-easycla bot commented Nov 9, 2023

CLA Signed

The committers listed above are authorized under a signed CLA.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. label Nov 9, 2023
@k8s-ci-robot
Copy link
Contributor

Welcome @GoutamVerma!

It looks like this is your first PR to kubernetes/website 🎉. Please refer to our pull request process documentation to help your PR have a smooth ride to approval.

You will be prompted by a bot to use commands during the review process. Do not be afraid to follow the prompts! It is okay to experiment. Here is the bot commands documentation.

You can also check if kubernetes/website has its own contribution guidelines.

You may want to refer to our testing guide if you run into trouble with your tests not passing.

If you are having difficulty getting your pull request seen, please follow the recommended escalation practices. Also, for tips and tricks in the contribution process you may want to read the Kubernetes contributor cheat sheet. We want to make sure your contribution gets all the attention it needs!

Thank you, and welcome to Kubernetes. 😃

@k8s-ci-robot k8s-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Nov 9, 2023
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign onlydole for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found 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 area/blog Issues or PRs related to the Kubernetes Blog subproject language/en Issues or PRs related to English language sig/docs Categorizes an issue or PR as relevant to SIG Docs. labels Nov 9, 2023
Copy link

netlify bot commented Nov 9, 2023

Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit dee1847
🔍 Latest deploy log https://app.netlify.com/sites/kubernetes-io-main-staging/deploys/658e8ee0db43d400074c001c
😎 Deploy Preview https://deploy-preview-43863--kubernetes-io-main-staging.netlify.app
📱 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 configuration.

@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 Nov 9, 2023
Copy link
Member

@aj11anuj aj11anuj left a comment

Choose a reason for hiding this comment

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

At some places unnecessary space lines can be removed to make whole blog look consistent. And some paragraphs are a little long, it will be better if those para will be braked into smaller lines for better text wrapping.

@dipesh-rawat
Copy link
Member

/retitle Add blog post about Storage in Kubernetes

@k8s-ci-robot k8s-ci-robot changed the title blog: storage in k8s Add blog post about Storage in Kubernetes Nov 10, 2023
Copy link
Member

@dipesh-rawat dipesh-rawat 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 the PR!

/hold
Pending assignment of publication date.

@k8s-ci-robot k8s-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Nov 10, 2023
Copy link
Member

Choose a reason for hiding this comment

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

Please try to incorporate diagrams in SVG format, preferably in line with our guide.

Copy link
Author

Choose a reason for hiding this comment

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

I have designed this diagram in Excalidraw. I've tried couple of online converter "png" to "svg" but they are not generating desired output.

Copy link
Contributor

@sftim sftim 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 the PR.

However, this isn't ready to publish and I have a strong concern that you didn't test any of the example code. Can you account for how the errors in the article have crept in?

content/en/blog/_posts/storage in k8s /index.md Outdated Show resolved Hide resolved
@GoutamVerma
Copy link
Author

Thanks for the PR.

However, this isn't ready to publish and I have a strong concern that you didn't test any of the example code. Can you account for how the errors in the article have crept in?

Yaml config in the image are sample test to understand how they simply connected with each other. Do I need to add the complete Yaml files for better understanding?

@sftim
Copy link
Contributor

sftim commented Nov 16, 2023

Yaml config in the image are sample test to understand how they simply connected with each other. Do I need to add the complete Yaml files for better understanding?

No, but the snippets need to be excerpts from valid manifests.

@sftim
Copy link
Contributor

sftim commented Nov 28, 2023

@GoutamVerma, to move this forward, would you be willing to explain how you've checked that the advice and comments in this article are technically accurate?

@GoutamVerma
Copy link
Author

@GoutamVerma, to move this forward, would you be willing to explain how you've checked that the advice and comments in this article are technically accurate?

Soon I will push the commit with updated snippets.

Copy link
Member

@Gauravpadam Gauravpadam 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 this insightful article @GoutamVerma!

I've added some initial reviews to move the progress on this PR

Here are some of my observations:

  • We need to correct the directory naming as

content/en/blog/_posts/<publication-date>-storage_in_kubernetes/
(publication date will be provided later)

  • The images are ideally in .svg format

  • Some general conventions are provided in the suggestions

I'll be happy to review once again post these amendments

content/en/blog/_posts/storage in k8s /index.md Outdated Show resolved Hide resolved
content/en/blog/_posts/storage in k8s /index.md Outdated Show resolved Hide resolved
content/en/blog/_posts/storage in k8s /index.md Outdated Show resolved Hide resolved
content/en/blog/_posts/storage in k8s /index.md Outdated Show resolved Hide resolved
content/en/blog/_posts/storage in k8s /index.md Outdated Show resolved Hide resolved
content/en/blog/_posts/storage in k8s /index.md Outdated Show resolved Hide resolved
content/en/blog/_posts/storage in k8s /index.md Outdated Show resolved Hide resolved
content/en/blog/_posts/storage in k8s /index.md Outdated Show resolved Hide resolved
content/en/blog/_posts/storage in k8s /index.md Outdated Show resolved Hide resolved

{{< figure src="image-4.png" title="storage class and pvc" >}}

In conclusion, StorageClass in Kubernetes is a fundamental component that allows users to dynamically provision and manage storage resources for their applications. Now that we have covered StorageClass, let's delve into the eight key storage principles in Kubernetes. These principles are essential for understanding how storage works within a Kubernetes cluster and how to effectively manage and utilize storage resources for your applications.
Copy link
Member

Choose a reason for hiding this comment

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

Is this supposed to be continued?

@sftim
Copy link
Contributor

sftim commented Dec 27, 2023

@GoutamVerma, to move this forward, would you be willing to explain how you've checked that the advice and comments in this article are technically accurate?

This is, for me, a key question and I don't yet see an answer. @GoutamVerma could you respond?

GoutamVerma and others added 3 commits December 29, 2023 14:35
Co-authored-by: Gaurav Padam <[email protected]>

Update content/en/blog/_posts/storage in k8s /index.md

Co-authored-by: Gaurav Padam <[email protected]>

Update content/en/blog/_posts/storage in k8s /index.md

Co-authored-by: Gaurav Padam <[email protected]>

Update : content/en/blog/_posts/storage in k8s /index.md

Co-authored-by: Gaurav Padam <[email protected]>

Update content/en/blog/_posts/storage in k8s /index.md

Co-authored-by: Gaurav Padam <[email protected]>

Update content/en/blog/_posts/storage in k8s /index.md

Co-authored-by: Gaurav Padam <[email protected]>

Update content/en/blog/_posts/storage in k8s /index.md

Co-authored-by: Gaurav Padam <[email protected]>

Update content/en/blog/_posts/storage in k8s /index.md

Co-authored-by: Gaurav Padam <[email protected]>

Update content/en/blog/_posts/storage in k8s /index.md

Co-authored-by: Gaurav Padam <[email protected]>

Update content/en/blog/_posts/storage in k8s /index.md

Co-authored-by: Gaurav Padam <[email protected]>

Update content/en/blog/_posts/storage in k8s /index.md

Co-authored-by: Gaurav Padam <[email protected]>

Update content/en/blog/_posts/storage in k8s /index.md

Co-authored-by: Gaurav Padam <[email protected]>

Update content/en/blog/_posts/storage in k8s /index.md

Co-authored-by: Gaurav Padam <[email protected]>

Update content/en/blog/_posts/storage in k8s /index.md

Co-authored-by: Gaurav Padam <[email protected]>

Update content/en/blog/_posts/storage in k8s /index.md

Co-authored-by: Gaurav Padam <[email protected]>

Update content/en/blog/_posts/storage in k8s /index.md

Co-authored-by: Gaurav Padam <[email protected]>

Update content/en/blog/_posts/storage in k8s /index.md

Co-authored-by: Gaurav Padam <[email protected]>

Update content/en/blog/_posts/storage in k8s /index.md

Co-authored-by: Gaurav Padam <[email protected]>

Update content/en/blog/_posts/storage in k8s /index.md

Co-authored-by: Gaurav Padam <[email protected]>

Update content/en/blog/_posts/storage in k8s /index.md

Co-authored-by: Gaurav Padam <[email protected]>
@GoutamVerma
Copy link
Author

@GoutamVerma, to move this forward, would you be willing to explain how you've checked that the advice and comments in this article are technically accurate?

This is, for me, a key question and I don't yet see an answer. @GoutamVerma could you respond?

Hey @sftim, sorry for delay in response.
I have tested this stuff and config files in virtual k8s environment that I have created.


* Create as cluster resource
* Created via YAML file
* Kind: persistent volume
Copy link
Contributor

Choose a reason for hiding this comment

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

Is that API kind correct? It doesn't look right.



{{< note >}}
Depending on the storage type in the backend, some attributes in the specification will vary because they are specific to the storage details. In the official [Kubernetes documentation](/docs/concepts/storage/volumes/#volume-types), you can find a comprehensive list of over 25 storage backends that Kubernetes supports.
Copy link
Contributor

Choose a reason for hiding this comment

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

Over 25 backends? Can you provide a list of these @GoutamVerma? I'd like to check the count.

zone: us-central1-a
type: pd-standard
```
It's essential to note that each storage backend typically has its own provisioner, which Kubernetes provides internally. These internal provisioners are prefixed with "_kubernetes.io_" like the one shown here in the above example. However, for other storage types or custom setups, external provisioners may be necessary. These external provisioners must be explicitly selected and integrated into your StorageClass configuration.
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this still true? I thought that in-tree storage plugins were deprecated.

Copy link
Contributor

@sftim sftim left a comment

Choose a reason for hiding this comment

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

I'm afraid that this article doesn't align with the expectations we have around clarity and accuracy. Right now, we wouldn't publish it.

@GoutamVerma
Copy link
Author

I'm afraid that this article doesn't align with the expectations we have around clarity and accuracy. Right now, we wouldn't publish it.

Let's me reconfigure the content and again make a PR for this.

@GoutamVerma GoutamVerma closed this Jan 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/blog Issues or PRs related to the Kubernetes Blog subproject cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. language/en Issues or PRs related to English language sig/docs Categorizes an issue or PR as relevant to SIG Docs. 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.

6 participants