dynamic resource allocation concepts by pohly · Pull Request #37766 · kubernetes/website

pohly · 2022-11-07T19:48:43Z

This will add a concept description for dynamic resource allocation.

netlify · 2022-11-07T20:01:04Z

✅ Pull request preview available for checking

Built without sensitive environment variables

Name	Link
🔨 Latest commit	`1a5017a`
🔍 Latest deploy log	https://app.netlify.com/sites/kubernetes-io-main-staging/deploys/6369612cc37b4200080f03ea
😎 Deploy Preview	https://deploy-preview-37766--kubernetes-io-main-staging.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

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

krol3 · 2022-11-21T02:04:11Z

Hey @pohly , 1.26 Docs Lead here 👋 !

The deadline (November 15th) for PRs ready to review has passed.

Please complete #37766 and request a review as soon as possible.

cc: @cathchu

krol3 · 2022-11-21T02:05:49Z

@pohly Please change the PR against dev-1.26 branch

leonardpahlke · 2022-11-21T19:14:43Z

Hey @pohly 🙂 what is the status on this? We need to start with the documentation & review process ASAP. We cannot add this feature without documentation! Thanks

netlify · 2022-11-22T10:53:26Z

👷 Deploy Preview for kubernetes-io-vnext-staging processing.

Name	Link
🔨 Latest commit	`197de4f`
🔍 Latest deploy log	https://app.netlify.com/sites/kubernetes-io-vnext-staging/deploys/63849211c5777d00082e11a2

pohly · 2022-11-22T10:53:59Z

/assign @klueska

Please review.

pohly · 2022-11-22T10:55:51Z

+The new `resource.k8s.io/v1alpha1` {{< glossary_tooltip text="API group"
+term_id="api-group" >}} provides four new types:
+
+- ResourceClass: defines which resource driver handles a certain kind of


Ideally these should link to the API documentation (similar to /docs/reference/kubernetes-api/config-and-storage-resources/csi-storage-capacity-v1/. But I don't know yet whether the new API group will have such generated documentation and where it will land.

Do we need to configure that?

Maybe for now, link each of them to the section in the KEP that discusses them? Then we can update the links once they are they are available as above).

There's no good anchor in the KEP for these. They would all link to the "API" section. I prefer to keep this as is until we have some generated API documentation.

pohly · 2022-11-23T09:47:53Z

The generated documentation looks good to me. I made some minor improvement (use description list) and one fix (list the new page in the index) since my first push.

klueska

First pass

klueska

Second pass.

krol3 · 2022-11-28T01:27:42Z

Hi @pohly. This PR needs a doc review by Mon Nov 28th to get this into the release. Please reach out to required SIGs to get their review. Thank you!

klueska · 2022-11-28T10:21:47Z

+  spec:
+    resourceClassName: resource.example.com
+    parametersRef:
+      apiGroup: cats.resource-driver.example.com


This needs to be updated to match above.

The documentation intentionally only gives an overview. The API documentation and the KEP have more details. Specific examples will be provided by hardware vendors that provide resource drivers. The new page gets placed between "Scheduling Framework" (because that introduces the concept of a scheduler plugin) and "Scheduler Performance Tuning" (which seems like a more advanced topic).

klueska · 2022-11-28T12:35:21Z

/lgtm
/approve

k8s-ci-robot · 2022-11-28T12:35:27Z

LGTM label has been added.

Details

Git tree hash: b62c7926d75076016296d222b4f7ea5a57ec1c3b

krol3

/lgtm

k8s-ci-robot · 2022-11-28T12:42:24Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: klueska, krol3

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

The pull request process is described here

Details

Needs approval from an approver in each of these files:

~~content/en/OWNERS~~ [krol3]

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 added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Nov 7, 2022

k8s-ci-robot requested a review from mehabhalodiya November 7, 2022 19:49

k8s-ci-robot added the language/en Issues or PRs related to English language label Nov 7, 2022

k8s-ci-robot requested a review from natalisucks November 7, 2022 19:49

k8s-ci-robot added the sig/docs Categorizes an issue or PR as relevant to SIG Docs. label Nov 7, 2022

pohly mentioned this pull request Nov 7, 2022

DRA: control plane controller ("classic DRA") kubernetes/enhancements#3063

Closed

42 tasks

k8s-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Nov 7, 2022

pohly changed the base branch from main to dev-1.26 November 21, 2022 09:25

pohly force-pushed the dynamic-resource-allocation-concepts branch from 1a5017a to 12f5290 Compare November 22, 2022 10:53

k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Nov 22, 2022

pohly changed the title ~~WIP: dynamic resource allocation concepts~~ Nov 22, 2022

k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 22, 2022

k8s-ci-robot assigned klueska Nov 22, 2022

pohly commented Nov 22, 2022

View reviewed changes

pohly force-pushed the dynamic-resource-allocation-concepts branch from 12f5290 to 1185ea0 Compare November 23, 2022 08:49

klueska reviewed Nov 23, 2022

View reviewed changes

pohly force-pushed the dynamic-resource-allocation-concepts branch from fe43be5 to ca5c83a Compare November 23, 2022 15:16

klueska reviewed Nov 24, 2022

View reviewed changes

Comment thread content/en/docs/concepts/scheduling-eviction/dynamic-resource-allocation.md Outdated

swatisehgal mentioned this pull request Nov 25, 2022

blog: node: device manager graduation to GA #37466

Merged

pohly force-pushed the dynamic-resource-allocation-concepts branch from 5f42d49 to 895a012 Compare November 28, 2022 07:37

klueska reviewed Nov 28, 2022

View reviewed changes

Comment thread content/en/docs/concepts/scheduling-eviction/dynamic-resource-allocation.md Outdated

Comment thread content/en/docs/concepts/scheduling-eviction/dynamic-resource-allocation.md

pohly force-pushed the dynamic-resource-allocation-concepts branch from b75e21d to 0bd79fc Compare November 28, 2022 10:17

klueska reviewed Nov 28, 2022

View reviewed changes

pohly force-pushed the dynamic-resource-allocation-concepts branch from 0bd79fc to 197de4f Compare November 28, 2022 10:48

k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Nov 28, 2022

krol3 approved these changes Nov 28, 2022

View reviewed changes

k8s-ci-robot assigned krol3 Nov 28, 2022

k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 28, 2022

k8s-ci-robot merged commit be09333 into kubernetes:dev-1.26 Nov 28, 2022

k8s-ci-robot added this to the 1.26 milestone Nov 28, 2022

klueska mentioned this pull request Dec 2, 2022

Add release announcement blog article for Kubernetes v1.26 #38123

Merged

Uh oh!

Conversation

pohly commented Nov 7, 2022

netlify Bot commented Nov 7, 2022

✅ Pull request preview available for checking

krol3 commented Nov 21, 2022

krol3 commented Nov 21, 2022

leonardpahlke commented Nov 21, 2022

netlify Bot commented Nov 22, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

👷 Deploy Preview for kubernetes-io-vnext-staging processing.

pohly commented Nov 22, 2022

pohly Nov 22, 2022

Choose a reason for hiding this comment

klueska Nov 24, 2022

Choose a reason for hiding this comment

pohly Nov 28, 2022

Choose a reason for hiding this comment

pohly commented Nov 23, 2022

klueska left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

klueska left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

krol3 commented Nov 28, 2022

Uh oh!

Uh oh!

klueska Nov 28, 2022

Choose a reason for hiding this comment

pohly Nov 28, 2022

Choose a reason for hiding this comment

klueska commented Nov 28, 2022

k8s-ci-robot commented Nov 28, 2022

krol3 left a comment

Choose a reason for hiding this comment

k8s-ci-robot commented Nov 28, 2022

Labels

5 participants

netlify Bot commented Nov 22, 2022 •

edited

Loading