-
Notifications
You must be signed in to change notification settings - Fork 376
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
Human-readable specification of Tracing Policy API #2152
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for tetragon ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
I see several errors in the deploy preview and will update this PR accordingly. Please defer a review until that's fixed. |
I converted to draft |
I've looked a bit further into this and it seems to me that the Markdown is somehow compiled by Netlify into its own app format. (I wasn't aware of Netlify's role when I set out on this task.) While the Markdown looks legit to me, not everything seems to translate and hence the rendering differs from what I had expected. Also perhaps additional metadata (between Current assessment: perhaps this needs attention from somebody with more Netlify experience than I can muster. |
mmh to my understanding (and hopefully I should get it because I set up the whole docs thingie) Netlify should not change anything to the website, everything is compiled statically using Hugo and the Docsy theme. You should have exactly the same result locally, using |
b6b0972
to
2f5a049
Compare
@mtardy I've by now gained a better understanding of how Hugo, Docsy and Netlify serve combined in the context of Tetragon and as a result have given this PR another try. One can now navigate between the generated web pages on the deploy preview site (see Documentation / Reference / Tracing Policy API). Could you guys perhaps have a look at your convenience to assess whether it makes sense to pursue this further? There is quite a lot of detailed documentation gleaned from the source code (perhaps it's too much). If the overall content seems fitting, the final goal IMO should be to automate the entire pipeline for generating it as part of |
could you maybe put everything on the same page so it's searchable? Plus you have a tons of pages without frontmatter that are messing up with the generation here. That could solve both issues. |
@mtardy re single page: OpenAPI Generator, the tool I've been using, apparently has three relevant (documentation) generators: re Hugo front matter: I am currently inserting front matter only on the single page markdown page |
* switching to generator html (i.e. single HTML page) * still work in progress This commit supports the generation of a human-readable specifition of the Tracing Policy API from source-code comments. fixes cilium#2074 Signed-off-by: Christian Hörtnagl <[email protected]>
@mtardy since you suggested putting everything on the same page I have switched generators from The are two new stages in I had to change OpenAPI Generator turned out to be quite a "Java monster" (it apparent serves OpenAPI code generation primarily, not OpenAPI documentation generation alone), and one could contemplate replacing it with another OpenAPI documentation generator in stage For now, I have included the generated HTML page |
This commit includes initial markdown documentation for the Tracing Policy API, which was generated by OpenAPI Generator. It is intended as 4th "tile" in the reference section of the Tetragon online documentation.
fixes #2074