Skip to content

Reorganize DRA subfeatures under Additional DRA features section#55311

Open
yashrajshuklaaa wants to merge 1 commit intokubernetes:mainfrom
yashrajshuklaaa:reorganize-dra-subfeatures
Open

Reorganize DRA subfeatures under Additional DRA features section#55311
yashrajshuklaaa wants to merge 1 commit intokubernetes:mainfrom
yashrajshuklaaa:reorganize-dra-subfeatures

Conversation

@yashrajshuklaaa
Copy link
Copy Markdown
Contributor

Description

Reorganize DRA subfeatures documentation to remove stage-based
section headings (## DRA beta features, ## DRA alpha features)
and consolidate under a single ## Additional DRA features section.

Feature stage is already conveyed per-subsection via the
{{< feature-state >}} shortcode, so the stage-named buckets add
churn without benefit — every graduation requires moving content
and updating anchors.

Also fixes Device Taints description which incorrectly said
alpha — it graduated to beta in 1.36. Demotes ## Consumable capacity
to ### to match the section hierarchy.

Issue

Closes: #55293

- Replace '## DRA beta features' and '## DRA alpha features' headings
  with a single '## Additional DRA features' section
- Remove stage-based intro paragraphs; feature state is conveyed by
  the existing feature-state shortcodes in each subsection
- Fix Device Taints description from alpha to beta (graduated in 1.36)
- Demote '## Consumable capacity' to '###' to match section hierarchy

Signed-off-by: Yashraj Shukla <shuklayashraj68@gmail.com>
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Apr 10, 2026
@k8s-ci-robot
Copy link
Copy Markdown
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 dipesh-rawat for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details 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 requested a review from klueska April 10, 2026 21:34
@k8s-ci-robot k8s-ci-robot added the language/en Issues or PRs related to English language label Apr 10, 2026
@k8s-ci-robot k8s-ci-robot requested a review from pohly April 10, 2026 21:34
@k8s-ci-robot k8s-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Apr 10, 2026
@netlify
Copy link
Copy Markdown

netlify bot commented Apr 10, 2026

Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit 65bf6be
🔍 Latest deploy log https://app.netlify.com/projects/kubernetes-io-main-staging/deploys/69d96cd45f07f0000845a253
😎 Deploy Preview https://deploy-preview-55311--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.

Copy link
Copy Markdown
Member

@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.

Looks nice.

Idea: move https://k8s.io/docs/concepts/scheduling-eviction/dynamic-resource-allocation/#prioritized-list into the advanced features section too.

@yashrajshuklaaa
Copy link
Copy Markdown
Contributor Author

Thank you for the review @lmktfy
Regarding the suggestion to move #prioritized-list - I can do that in this PR if preferred, or open a follow-up to keep this PR minimal and focused. Please let me know which you'd prefer and I'll proceed accordingly

@pohly
Copy link
Copy Markdown
Contributor

pohly commented Apr 11, 2026

Didn't I already do exactly this in #54648 for the dev-1.36 branch? Wouldn't it be simpler to wait until the 1.36 docs get merged into main?

/hold

@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 Apr 11, 2026
@pohly
Copy link
Copy Markdown
Contributor

pohly commented Apr 11, 2026

Regarding "prioritized list": it's debatable whether it's an advanced feature or something that normal users might want to know about in the section which describes how to request resources. Personally I prefer it where it is.

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. 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 size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Re-organize documentation of beta and alpha DRA subfeatures

4 participants