-
Notifications
You must be signed in to change notification settings - Fork 163
upgrade docs signifficant refinement #2296
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
Conversation
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.
LGTM! 🚀
Really, really nice! I added just a few small suggestions.
Co-authored-by: David Kilfoyle <41695641+kilfoyle@users.noreply.github.com>
Co-authored-by: David Kilfoyle <41695641+kilfoyle@users.noreply.github.com>
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 update looks really nice and I think it is a big improvement. mostly commented style things but there are a couple of questions peppered in.
a couple of overall takeaways:
- do we think these docs all make sense for someone who has to upgrade from 8.18? just want to make sure the paths are solid.
- the procedures are quite dense and could possibly benefit from the stepper component, but that's not a problem for now
...oy-manage/upgrade/deployment-or-cluster/reading-indices-from-older-elasticsearch-versions.md
Show resolved
Hide resolved
Co-authored-by: shainaraskas <58563081+shainaraskas@users.noreply.github.com>
Co-authored-by: shainaraskas <58563081+shainaraskas@users.noreply.github.com>
Minor issues & leftovers after #2296
This PR include multiple changes and improvements in the upgrade docs:
Upgrade orchestrators moved downward in the TOC
Landing page (upgrade.md): Improved introduction, added an overview upgrade section as a map for the entire upgrade documentation, and upgrade-path to highlight that important information.
Plan upgrade: Scope clarified, the page is about general recommendations for the upgrade and compatibility verifications in order to create an upgrade plan. Added an
upgrade order
section, and a finalexample
.Prepare to upgade: We had lost part of the content (minor upgrades preparation steps) during the docs migration, and now that we have multiple 9.x versions that part feels needed. I've also re-worked that page and put all common steps (for major or minor upgrades) together, as the original asciidoc document (https://www.elastic.co/guide/en/elastic-stack/8.18/upgrading-elastic-stack.html) also felt repetitive and a bit duplicate.
Upgrade your deployment or cluster: changed a bit the intro and structure, plus added
out of order
releases section, related and linked from the upgrade-paths of the landing page.Upgrade Elasticsearch self-managed (rolling upgrade instructions): Just minor changes adding some headings for better visibility.
I've tried to also position properly the documents in the flow, with "next steps" sections and proper introductions.
Closes: #1608
Part of the work of #1685