Skip to content

Conversation

@jrodewig
Copy link
Contributor

******
The Upgrade Assistant and migration APIs are enabled with both the Basic and
Trial licenses. You can install the default distribution of 6.7 to use the
Trial licenses. You can install the default distribution of 6.8 to use the
Copy link

Choose a reason for hiding this comment

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

Is the deletion of trial licenses relevant here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

From what I understand, trial licenses can still be used for machine learning features.

{ref}/reindex-upgrade-remote.html[reindex from remote].

The recommended path is to upgrade to 6.7 before upgrading to {version}. This
The recommended path is to upgrade to 6.8 before upgrading to {version}. This
Copy link

Choose a reason for hiding this comment

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

Just wonder if this is strong enough language around 'we recommend you upgrade from 6.8 to 7.1' due to changed security licensing. (I admit, I'm having trouble wrapping my brain around the recommended upgrade path.)

Choose a reason for hiding this comment

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

I don't think the changes to license changes the recommendations here. Here it is recommended so you can do a rolling restart (as opposed to a full cluster restart) which was true before and after the license change.

Copy link
Contributor

Choose a reason for hiding this comment

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

The recommended upgrade path is through the last minor of the previous major to the most recent minor. So now 6.8 -> 7.1.

Copy link

@jakelandis jakelandis left a comment

Choose a reason for hiding this comment

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

LGTM

@jrodewig jrodewig merged commit 04d37ed into elastic:master May 20, 2019
@jrodewig jrodewig deleted the fix-6.7-assumptions branch May 20, 2019 20:42
@jrodewig
Copy link
Contributor Author

jrodewig commented May 20, 2019

Backported to 7.x and 7.1 on ac97379.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants