-
Notifications
You must be signed in to change notification settings - Fork 14.5k
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
update generated api-reference docs for 1.10 #7856
Conversation
Deploy preview for kubernetes-io-vnext-staging ready! Built with commit 9e4cb50 https://deploy-preview-7856--kubernetes-io-vnext-staging.netlify.com |
/lgtm |
@Bradamant3 Initial pass: It doesn't seem right that we have several cases of "File renamed without changes". I'll look a little deeper. |
@steveperry-53 you've been through this before, so I'll defer to you. But when I saw those "File renamed without changes" diffs, I also noticed that they are all css/js files for presenting the content. They aren't content files themselves. So I'd sort of expect them to remain the same. |
Apparently I missed a critical step in the docs: creating a new directory to receive the copied files. In my local kubernetes/website/docs/reference/generated/kubernetes-api, Now back in my local kubernetes-incubator/reference-docs, That gives me these new files in my local kubernetes/website:
|
hrm. OK, running various things again. bc kubectl docs also messed up. |
@Bradamant3 Given that you deleted the 1.9 files, I think this PR is correct. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: steveperry-53, zacharysarah 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 |
/hold cancel |
…elease-1.10 * 'master' of https://github.com/kubernetes/website: Update overview.md (#7836) Consolidate all upgrade/downgrade admin tasks (#7735) add generated docs for tools and components (#7845) (#7866) update generated api-reference docs for 1.10 (#7856) (#7864) Merge 1.10 to master for release (#7861)
Also updated the _data/reference.yml file, in addition to the generated docs.
It looks as though the issue with the make copyapi -- or possibly with cp, haven't checked that far -- is that it does not like to create directory names with punctuation in them. Or least, not with either dashes or periods. Not fully tested yet, not sure what to recommend going forward. Will ponder later. Thanks for all your help.
Addendum: the previous is not correct. Or at least, it does not appear to be an issue with the generated kubectl docs.
/assign @steveperry-53