Skip to content

Conversation

tengqm
Copy link
Contributor

@tengqm tengqm commented Oct 5, 2025

This PR moves the ephemeral storage related configuration out of the container resource management page. This proposal is based on the following observations:

  • the current container resource management page is too long (1000+ lines) for maintenance.
  • the ephemeral storage section is well structured and self-contained.
  • the ephemeral storage section itself is 300+ lines, suitable for a dedicated page.
  • the configuration of ephemeral storage is a complicated topic in and by itself -- it involves operations for admins (i.e. one FS vs two FS) and for app developers (i.e. container level requests).
  • the management of local ephemeral storage is a pain point for many cluster operators. We may envision more details to be added to this topic.

@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 urvashi0109 for approval. For more information see the 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 language/en Issues or PRs related to English language cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Oct 5, 2025
@k8s-ci-robot k8s-ci-robot added the size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. label Oct 5, 2025
Copy link

netlify bot commented Oct 5, 2025

Pull request preview available for checking

Name Link
🔨 Latest commit eea493b
🔍 Latest deploy log https://app.netlify.com/projects/kubernetes-io-main-staging/deploys/68e2fdd951195d00088b8af6
😎 Deploy Preview https://deploy-preview-52628--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 project configuration.

@lmktfy
Copy link
Contributor

lmktfy commented Oct 5, 2025

/label refactor

@k8s-ci-robot k8s-ci-robot added the refactor Indicates a PR with large refactoring changes e.g. removes files or moves content label Oct 5, 2025
Copy link
Contributor

@lmktfy lmktfy 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 spotting this.

I very strongly recommend moving the page about Local ephemeral storage to live within https://kubernetes.io/docs/concepts/storage/

@tengqm
Copy link
Contributor Author

tengqm commented Oct 5, 2025

Thanks for spotting this.

I very strongly recommend moving the page about Local ephemeral storage to live within https://kubernetes.io/docs/concepts/storage/

Makes sense. Revised.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/en Issues or PRs related to English language refactor Indicates a PR with large refactoring changes e.g. removes files or moves content size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants