-
Notifications
You must be signed in to change notification settings - Fork 14k
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
Remove link from Device Plugins page to KEP #41145
Conversation
✅ Pull request preview available for checkingBuilt without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
@@ -10,7 +10,7 @@ weight: 20 | |||
<!-- overview --> | |||
{{< feature-state for_k8s_version="v1.26" state="stable" >}} | |||
|
|||
Kubernetes provides a [device plugin framework](https://git.k8s.io/design-proposals-archive/resource-management/device-plugin.md) | |||
Kubernetes provides a [device plugin framework](https://git.k8s.io/enhancements/keps/sig-node/3573-device-plugin/README.md) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We shouldn't link to https://git.k8s.io/enhancements/keps/sig-node/3573-device-plugin/README.md with this link text, as that page doesn't mention the word framework anywhere. This page itself is about device plugins.
I would omit the link here. We can link to https://kep.k8s.io/3573 in the What's next section at the foot of the page, if we want to.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, yeah, makes sense.
Not sure what text would suit the link in the What's next section
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about removing the hyperlink?
/retitle point Device Plugins page to KEP |
/retitle Remove link from Device Plugins page to KEP @omBratteng please rebase this against main (see https://lwkd.info/2023/20230606#developer-news for context). If you're not sure how to, please reply here to say so and we'll try to help. |
If you're willing to rebase rather than merge in main, we'd prefer that. |
I did a rebase after I saw that I first had merged main |
c2bc277
to
b3ef78a
Compare
Just rebased to get the last commit on main as well |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/label tide/merge-method-squash
LGTM label has been added. Git tree hash: cbcb3ffa96fab2303a7eb6b9b7157837abd9f60b
|
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: reylejano The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Update the link to the proposal on the Device Plugins page. Previously pointed to archived repository, now points to enhancements repo and the specific KEP