🌱 Auto-generate API documentation from CRDs#2605
🌱 Auto-generate API documentation from CRDs#2605MahnoorAsghar wants to merge 1 commit intometal3-io:mainfrom
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
This is copy from IRSO's api doc generator right? |
Yes, it is |
2a53530 to
fbed203
Compare
|
/retest-required |
|
/retest-required |
tuminoid
left a comment
There was a problem hiding this comment.
The difference between this BMO PR and IRSO here is that IRSO runs the "make manifests" as part of build Github workflow (in a VM), while BMO runs make generate as a Prow job (in a container).
The correct way for BMO would be to have it in a separate Prow job, which runs the crdoc container in Prow.
hack/gen-api-doc.sh
Outdated
| "${CONTAINER_RUNTIME}" run --rm -v "${PWD}":/src:ro,z \ | ||
| ghcr.io/fybrik/crdoc:"${CRDOC_VERSION}" \ | ||
| --resources /src/config/base/crds/bases/ --output /dev/stdout \ | ||
| > docs/api.md No newline at end of file |
fbed203 to
e87422f
Compare
e87422f to
6b1e0cb
Compare
|
I dont remember how docs/api-overview.md came from, it must be output from the script, but executing it right now does not create it. |
1d6aabf to
861dc18
Compare
Signed-off-by: Mahnoor Asghar <masghar@redhat.com>
861dc18 to
05b5230
Compare
|
For the check-links-pr job, https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.24/#objectmeta-v1-meta is the correct url, but the crdoc container is generating v1.20 links which are indeed broken This could be because the latest crdoc container version is 3 years old. There are 2 tagged releases on the repo after that, but no associated container images. See fybrik/crdoc#322 |
|
I could add the exclude-path option to https://github.com/metal3-io/baremetal-operator/blob/main/.github/workflows/pr-link-check.yml and https://github.com/metal3-io/project-infra/blob/1f19a2b8b926ab9bec62b1b8099cecf492d2125d/.github/workflows/pr-link-check.yml#L64 |
|
PR needs rebase. 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. |
Auto-generate API documentation from CRDs
Assisted-by: Claude
Fixes #2509