-
Notifications
You must be signed in to change notification settings - Fork 365
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
Invalid api-conventions URL link #258
Invalid api-conventions URL link #258
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: c3y1huang The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Hi @c3y1huang. Thanks for your PR. I'm waiting for a kubernetes-csi 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. Instructions 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/test-infra repository. |
@@ -27,7 +27,8 @@ spec: | |||
apiVersion: | |||
description: 'APIVersion defines the versioned schema of this representation | |||
of an object. Servers should convert recognized schemas to the latest | |||
internal value, and may reject unrecognized values. More info: https://git.k8s.io/community/contributors/devel/api-conventions.md#resources' | |||
internal value, and may reject unrecognized values. | |||
More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#resources' |
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.
Those are generated automatically by running hack/update-crd.sh so we are not supposed to modify them manually.
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.
Reverted URL update for CRDs.
6b9d654
to
50ed4fd
Compare
@yuxiangqian PTAL. |
CI failed because you also have to update generated code after modifying types.go. Take a look of the following issue. Generating code is getting complicated after we changed the package path for v2: |
50ed4fd
to
b2239a2
Compare
Signed-off-by: Chin-Ya Huang <chin-ya.huang@suse.com>
b2239a2
to
a8c587a
Compare
Thanks @xing-yang , CI failed due to vendor changes. Don't know what I was thinking :-/. Revert to address. |
Just noticed that this other PR (#256) which was submitted a few days earlier than this PR already contains changes in this PR. Let's put this PR on hold for now, wait until the other PR is merged, and then see if there're any additional things that need to be addressed. |
@c3y1huang the PR changs intended here already got in via #260 , unless if some other changes are planned with this PR, I believe we could close this one.. Thanks! @xing-yang what do you think ? |
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
Stale issues rot after 30d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
@c3y1huang: PR needs rebase. Instructions 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/test-infra repository. |
What type of PR is this?
/kind cleanup
/kind documentation
What this PR does / why we need it:
Original URL link is invalid.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?: