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
Fix cross-repo link #47223
Fix cross-repo link #47223
Conversation
Not sure there is a better solution. The link must work inside of staging and in the client-go repo. Making in-project+cross-project links seems to be impossible. |
@sttts still 404 for me: https://github.com/kubernetes/apiextensions-apiserver/ Is this repo private? |
@ahmetb it doesn't exist yet ;-) Compare by kubernetes-dev mail from yesterday. |
@@ -4,4 +4,4 @@ | |||
|
|||
For a client-go example using CustomResourceDefinitions, go to | |||
|
|||
http://github.com/kubernetes/apiextensions-apiserver/examples/client-go. | |||
[k8s.io/apiextensions-apiserver/examples/client-go](https://github.com/kubernetes/apiextensions-apiserver/tree/master/examples/client-go). |
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.
@thockin created https://git.k8s.io for stuff like this.
Try https://git.k8s.io/apiextentions-apiserver/examples/client-go
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.
that's a dead link right now...
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.
done
This example probably should not have been merged in the first place if the corresponding repository doesn't exist yet? 😕 |
baf6dee
to
2aa87e4
Compare
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.
This looks good to me, except that kubernetes/apiextensions-apiserver
is empty. Once there is content here, then this can be merged.
Repo exists, but is still empty. @caesarxuchao is updating the bot now. |
/retest |
pull-kubernetes-federation-e2e-gce is flaky: #47672 |
OK, there's content here https://github.com/kubernetes/apiextensions-apiserver |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: spiffxp, sttts Associated issue: 47211 The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
Automatic merge from submit-queue |
@@ -4,4 +4,4 @@ | |||
|
|||
For a client-go example using CustomResourceDefinitions, go to | |||
|
|||
http://github.com/kubernetes/apiextensions-apiserver/examples/client-go. | |||
[k8s.io/apiextensions-apiserver/examples/client-go](https://git.k8s.io/apiextentions-apiserver/examples/client-go). |
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.
s/apiextentions/apiextensions. fix in #47684
Fixing #47211