-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
[docs] Migrate docs from AsciiDoc to Markdown #42897
base: main
Are you sure you want to change the base?
Conversation
Reviewers, please let me know if you need any details about this. I assume that the failing CI checks don't matter, or the tests can be removed since they won't be applicable any longer with the new Markdown docs. Also, just for awareness, we'll have a 1:1 mapping so that each HTML docs page now has a single Markdown source file. Previously, changes in the 'libbeat' directory were propagated to the Metricbeat docs, Filebeat docs, etc. We're making similar changes across all of the docs to reduce complexity. |
It's not currently possible to use a variable in a link, but I opened an issue with the docs engineering team. Feel free to add more details (or correct my description if needed!) in that issue. |
Thanks @colleenmcginnis, that captures it. |
Nice catch! I see those are all converted to .md so I'll remove them, but I see there are lots of other |
I confirmed that the asciidoc and image files under the One thing I'm confused about: These Generated docs instructions say that @belimawr Do you know if we still have generated docs? If that's still needed, I can open an issue to ask the Docs Engineering team about converting the generation scripts to output Markdown instead of Asciidoc. |
For now, I think these should remain as is. The Beats Release Notes are generated automatically so we'll need to get that process fixed up to generate Markdown. As a temporary fix, the 9.0 Beta Release Notes (in Asciidoc) were copied over into the Stack Docs repo, here. |
I'm not sure... But I'll investigate and get back to you once I have an answer. |
This pull request is now in conflicts. Could you fix it? 🙏
|
@kilfoyle, yes, we still have generated docs and as far as I can tell we still need them. While the Generated docs is a bit outdated (e.g: So we have to update them to output Markdown and to stop relying on reading versions from |
Thanks @belimawr! If I should open an issue for those changes or if there's anything else I can do, please let me know. |
Yes, please, could you open an issue for that? Given your comment:
I understood you'd open the issue. Honestly, I'm not sure who is responsible for those scripts: the Data-Plane (Beats) team or the Docs team 🙈. Anyway, if there is anything I can help with, just ping me ;) |
Unfortunately, the Docs Eng team doesn’t have the bandwidth to update automation scripts for every engineering team 😞 . We know this work takes time to prioritize and schedule, so we can ship static docs (this point-in-time snapshot) for teams that need more runway while they update their automation. Ideally, we'd be able to do that from this repo. But it sounds like there’s some automation that needs fixing before we can merge this PR. @kilfoyle is meeting with @pierrehilbert on Friday. I've asked to be added to that meeting to continue the discussion. |
Thanks a lot @belimawr for the investigation and @bmorelli25 for the clarification! And sorry for my confusion: I wasn't sure which team should or could own this work. I've opened #43008 to see if we can:
If I've missed any info in the issue please feel free to make any additions/corrections. On Friday, Brandon and I will chat with Pierre about this issue and also the Beats Release Notes. |
This pull request is now in conflicts. Could you fix it? 🙏
|
Migrate docs from AsciiDoc to Markdown.
Please see docs preview
Notes:
{beat}/docs/
directory.For reference, here are details about the docs migration.