projectsyn / lieutenant-operator

The Project Syn Inventory API Operator
https://docs.syn.tools/lieutenant-operator/
BSD 3-Clause "New" or "Revised" License
3 stars 1 forks source link

Write Documentation #37

Closed srueg closed 4 years ago

srueg commented 4 years ago

Write some basic documentation about this tool. Reuse as much as possible from the SDD: https://github.com/projectsyn/documentation/blob/master/docs/modules/SDDs/pages/0007-lieutenant-management-api.adoc If possible, generate documentation for the OpenAPI CRDs (e.g. using gen-crd-api-reference-docs) and include it in the Antora docs.

Task Deliverables

Kidswiss commented 4 years ago

I added a bit of a hacky script togenerate.go which uses your linked generator to generate a html file for our CRDs. That file in turn is included in the asciidoc documentation.

I also transferred everything from the SDD to this repo. The CRD topics are now a bit redundant. Should I remove the SDD CRD definitions in favour of the concrete implementations? @srueg

srueg commented 4 years ago

Nice! Yes good point. I think only leaving the generated part is ok.

Kidswiss commented 4 years ago

Implemented your feedback and removed the old CRD documentation.

Kidswiss commented 4 years ago

Unfortunately the html include does not work currently: https://docs.syn.tools/lieutenant-operator/crds-html.html

Will investigate

srueg commented 4 years ago

@akosma can you please add the Makefile setup for Antora to this repo (and also include it in the GitHub action)?

Kidswiss commented 4 years ago

After quite some investigation I found two things:

Lokal renders look good now: image

@srueg Can you have a look and merge if everything looks good to you? THX

srueg commented 4 years ago

Looking good 👍 https://syn.tools/lieutenant-operator/crds-html.html

Kidswiss commented 4 years ago

Nice!

The docs now look good, closing here.