Skip to content
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.

Sign up for GitHub

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] Simplify ML upgrade step #40006

Merged
merged 6 commits into from
Mar 25, 2019
Merged

[DOCS] Simplify ML upgrade step #40006

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
67b1447
[DOCS] Simplify ML upgrade step
lcawl Mar 13, 2019
445550d
[DOCS] Edits machine learning upgrade steps
lcawl Mar 14, 2019
faec5f4
[DOCS] Adds upgrade mode option to reindex scenario
lcawl Mar 14, 2019
675419f
[DOCS] Reverts changes to versions
lcawl Mar 14, 2019
521a3b3
[DOCS] Adds ML advice to reindex from remote
lcawl Mar 19, 2019
4ff8ed4
[DOCS] Add info about preventing inconsistencies
lcawl Mar 25, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion docs/reference/upgrade/cluster_restart.asciidoc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ recovery.
include::synced-flush.asciidoc[]
--

. *Stop any machine learning jobs that are running.* (Optional)
. *Temporarily stop the tasks associated with active {ml} jobs and {dfeeds}.* (Optional)
+
--
include::close-ml.asciidoc[]
Expand Down
8 changes: 8 additions & 0 deletions docs/reference/upgrade/reindex_upgrade.asciidoc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -112,6 +112,14 @@ indices from the previous major version to be upgraded to the
current major version. Skipping a major version means that you must
resolve any backward compatibility issues yourself.

ifdef::include-xpack[]
If you use {ml-features} and you're migrating indices from a 6.5 or earlier
cluster, the job and {dfeed} configuration information are not stored in an
index. You must recreate your {ml} jobs in the new cluster. If you are migrating from a 6.6 or later cluster, it is a good idea to temporarily halt the tasks associated with your {ml} jobs and {dfeeds} and prevent new jobs from opening
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The jobs won't open on the new cluster because whether jobs are open or closed is determined by persistent tasks that won't be brought across by the reindex.

So I would change "prevent new jobs from opening" to something like "prevent inconsistencies between different ML indices that are reindexed at slightly different times".

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @droberts195 does that clarification apply to the "reindex in place" section too?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For reindex in place the persistent tasks are in the same cluster, so I think that clarification only applies to reindex from remote.

during the reindex. Use the <<ml-set-upgrade-mode,set upgrade mode API>> or
{stack-ov}/stopping-ml.html[stop all {dfeeds} and close all {ml} jobs].
endif::include-xpack[]

=============================================

To migrate your indices:
Expand Down
2 changes: 1 addition & 1 deletion docs/reference/upgrade/rolling_upgrade.asciidoc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ include::synced-flush.asciidoc[]

--

. *Stop any machine learning jobs that are running.* (Optional)
. *Temporarily stop the tasks associated with active {ml} jobs and {dfeeds}.* (Optional)
+
--
include::close-ml.asciidoc[]
Expand Down