-
Notifications
You must be signed in to change notification settings - Fork 16
docs: migrate repository links from google to kubernetes-sigs org #46 #47
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
docs: migrate repository links from google to kubernetes-sigs org #46 #47
Conversation
|
Hi @mahmut-Abi. Thanks for your PR. I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
| ## DRANET + MPI Operator: A Powerful Combination | ||
|
|
||
| - DRANET: Provides the mechanism to discover RDMA-capable NICs on your Kubernetes nodes and make them available for Pods to claim. Through DRA, Pods can request a specific NIC, and DRANET, via NRI hooks, will configure it within the Pod's namespace, [even naming it predictably (e.g., dranet0)](google/dranet/dranet-dcd98f563b1a24f4800cf3d2d502ec5b2f488ddc/site/content/docs/user/interface-configuration.md) | ||
| - DRANET: Provides the mechanism to discover RDMA-capable NICs on your Kubernetes nodes and make them available for Pods to claim. Through DRA, Pods can request a specific NIC, and DRANET, via NRI hooks, will configure it within the Pod's namespace, [even naming it predictably (e.g., dranet0)](/docs/user/interface-configuration.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.
Not sure how this affects the site now that we are in k8s sigs, @aojea or @gauravkghildiyal do you know?
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 can leave this open and try and see
|
/ok-to-test |
|
/lgtm cancel hmm, we need to setup github pages here first |
|
I've added the cname for dranet.sigs.k8s.io kubernetes/k8s.io#9004 , we should try to set that up @mahmut-Abi , can you replace the dranet.dev config from the github actions and hugo config to use dranet.sigs.k8s.io |
Signed-off-by: maihemutimaimaiti <maihemutimaimaiti@360.cn>
|
@aojea done. |
|
/lgtm let's iterate |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aojea, mahmut-Abi The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
The code and tests were migrated to kubernetes-sigs organization, but the documentation was still referencing the old google/dranet repo.
This commit updates all documentation links to point to the new kubernetes-sigs/dranet repository to avoid user confusion.
Ref: #45