-
Notifications
You must be signed in to change notification settings - Fork 25.6k
Prep docs for v9 #115375
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
Prep docs for v9 #115375
Conversation
|
Documentation preview: |
|
Pinging @elastic/es-docs (Team:Docs) |
|
|
||
| endif::[] | ||
|
|
||
| // The notable-highlights tag marks entries that | ||
| // should be featured in the Stack Installation and Upgrade Guide: | ||
| // tag::notable-highlights[] | ||
|
|
||
| [discrete] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
AFAIK these are all 8.x release highlights.
| in the migration guide. | ||
| //TODO: figure out how to get the link below to work ! | ||
| //For information about specific breaking changes and the impact of requesting | ||
| //compatibility mode. See xref:/docs/reference/migration/migrate_9_0/rest-api-changes.asciidoc[REST API changes] in the migration guide. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jmikell821 - can you help me figure out how to make this reference work ?
I think we've updated things so we don't forward port 8.x release notes to |
| in the migration guide. | ||
| //TODO: figure out how to get the link below to work ! | ||
| //For information about specific breaking changes and the impact of requesting | ||
| //compatibility mode. See xref:/docs/reference/migration/migrate_9_0/rest-api-changes.asciidoc[REST API changes] in the migration guide. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| //compatibility mode. See xref:/docs/reference/migration/migrate_9_0/rest-api-changes.asciidoc[REST API changes] in the migration guide. | |
| //compatibility mode. See <<rest-api-changes,REST API changes>> in the migration guide. |
driving by here :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Liam, I tried that but got:
INFO:build_docs:asciidoctor: WARNING: invalid reference: rest-api-changes
The link is trying :
docs/reference/rest-api/rest-api-compatibility.asciidoc <- from here
docs/reference/migration/migrate_9_0/rest-api-changes.asciidoc <- to here
I only updated the file name and anchor name to be 9 instead of 8. This used to work : <<breaking_80_rest_api_changes, REST API changes>>
This also, does not work: See <<breaking_90_rest_api_changes,REST API changes>>
INFO:build_docs:asciidoctor: WARNING: invalid reference: breaking_90_rest_api_changes
So I assume that maybe I removed some implicit transitive include or something like that (I am pretty clueless with asciidoc)
EDIT: on second thought, there is no need to remove the change logs now, so I will leave them alone (reverted the removal) |
This reverts commit 0cd8cb2.
|
pull back to draft since I just realized this competes with #114154 that does the same thing (but slightly better). |
|
Closing this as dupcliate of #114154 |
|
re-opening since #114154 was closed |
|
@leemthompo - could you please review. the docs don't need to be perfect for now, but we need a place to start collecting the information. |
leemthompo
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couple questions, happy to approve if need unblocking to move forward, just re-ping :)
| @@ -0,0 +1,314 @@ | |||
| [[migrating-9.0]] | |||
| == Migrating to 9.0 | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jakelandis there's a lot of stuff in here that's already in 8.x, is that going to be more or less confusing to clean up now versus in a follow-up?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We will end fighting abit with the automation. So even if i clean this up now, it will just come back the next time someone generates the release notes. I think we will need to fix this is in the automation, or just go manual, or something in-between. I would prefer to address in a follow up since the changes here are intended to mechanical. I will add a comment to this file (and others) something to the effect that this is generated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I updated this PR with a comment for these generated file that links to an internal issue for us to follow up with. I also commented out the content created by the automation. While no one is probably looking at the v9 release notes (it's not released yet) it is better to have no information than wrong information.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯
| [chapter] | ||
| [[release-highlights]] | ||
| = What's new in {minor-version} | ||
| == What's new in {minor-version} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I presume the removal of [chapter] and the deepening of sub-headings was created by automation?
We recently changed the IA of the left-nav and we don't want to lose these.
tl;dr we need the chapter id and the headings here should start at H2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, this was created by automation. If if fix it now, it will come back the next time some generates the release notes. I will add a comment to this effect to this file too and will address in a follow up.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks!
| @@ -1,3 +1,6 @@ | |||
| // THIS IS A GENERATED FILE. DO NOT EDIT DIRECTLY. | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
❤️
This commit prepares the documentation for version 9. Some of the automation generates docs that are not correct for version 9. The content has been commented out with a reference to an internal issue for us to address before this documentation is used.
This commit prepares the docs for v9.
Note - Some of this content is generated by
./gradlew generateReleaseNotesand is may not be accurate to v9 only. We will need to manually adjust later. For now just trying to get the big change in.Also, the REST API doc is showing up in the transient setting migration guide ...I added a fixme to fix later. Again, just trying to get the big change in.