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.

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

Support config upgrades from Kibana v7.10.x directly to OpenSearch Dashboards v1.1 #741

Closed
shawnz opened this issue Aug 26, 2021 · 2 comments · Fixed by #743
Closed

Support config upgrades from Kibana v7.10.x directly to OpenSearch Dashboards v1.1 #741

shawnz opened this issue Aug 26, 2021 · 2 comments · Fixed by #743
Labels
enhancement New feature or request v1.1.0

Comments

@shawnz
Copy link

shawnz commented Aug 26, 2021

Hi there, I am writing to request explicit support for an upgrade path from Kibana v7.10.x directly to OpenSearch Dashboards v1.1.

I am not sure if this is already planned, but in this patchset #485 I noticed the following line of code:

const currentVersionIsVersion1 = semver.eq(opensearchDashboardsReleaseVersion, '1.0.0');

This seems to indicate that at the current time, only v1.0 will support that upgrade path.

The reason why I would like to be able to upgrade directly to v1.1 is because v1.0 will not be provided as an RPM package. Since we are currently using the ODFE RPM packages, it would be inconvenient if we could not upgrade directly to v1.1.

Further discussion:

#334 (comment)
opensearch-project/OpenSearch#671 (comment)

Thanks for reading, and my apologies if this is already planned.

Shawn

@shawnz shawnz added the enhancement New feature or request label Aug 26, 2021
@kavilla
Copy link
Member

kavilla commented Aug 26, 2021

Hello @shawnz, thanks for opening this.

Great catch, OpenSearch Dashboards should pretty much support direct upgrades from Kibana 7.x (verified for at least 7.10.2). The line of code you are referencing is basically for carrying over any advanced settings you have. Since what OpenSearch Dashboards is doing is checking your current version of the application and looking for any advanced settings that exist prior to it's current version, but since we went from a higher version to a lower version it wasn't able to find any original settings.

Ideally we were thinking you should have to go from a previous version to 1.0 first but that has changed. So if you go from Kibana 7.10.x to OpenSearch Dashboards 1.1, everything should be carried over it's just it won't directly apply your advanced settings if you have any set. For example, if you set dark mode in 7.10.x, you would expect to see dark mode set in OpenSearch Dashboards. Your advanced settings for 7.10.x will actually still exist and you can still it as a saved object so a workaround would be carrying over the values you set to the config for 1.1.0. But that is a bad experience so we should fix this to relax the version check. We definitely will want to keep this version check for at least 1.x, so we don't get in a weird state if OpenSearch does get to 7.10.x then this code might break some expected functionality.

@shawnz
Copy link
Author

shawnz commented Aug 27, 2021

Thank you for the notice! Good to know as we don't really rely on any Kibana advanced settings anyway.

kavilla added a commit to kavilla/OpenSearch-Dashboards-1 that referenced this issue Aug 27, 2021
An original update to enable taking settings from a valid legacy
version and applying to current OpenSearch Dashboards was made
here:

opensearch-project#485

However, it explicitly checked for current version being 1.0.0, which
is too strict because ideally all versions of 1.x is compatible.

This makes the config check more relaxed and will taking settings from
a legacy version if the current version is 1.x.

Issue resolved:
opensearch-project#741

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>
kavilla added a commit to kavilla/OpenSearch-Dashboards-1 that referenced this issue Aug 27, 2021
An original update to enable taking settings from a valid legacy
version and applying to current OpenSearch Dashboards was made
here:

opensearch-project#485

However, it explicitly checked for current version being 1.0.0, which
is too strict because ideally all versions of 1.x is compatible.

This makes the config check more relaxed and will taking settings from
a legacy version if the current version is 1.x.

Issue resolved:
opensearch-project#741

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>
@kavilla kavilla linked a pull request Aug 27, 2021 that will close this issue
5 tasks
@kavilla kavilla changed the title Support upgrades from Kibana v7.10.x directly to OpenSearch Dashboards v1.1 Support config upgrades from Kibana v7.10.x directly to OpenSearch Dashboards v1.1 Aug 27, 2021
@kavilla kavilla added the v1.1.0 label Aug 31, 2021
kavilla added a commit that referenced this issue Aug 31, 2021
An original update to enable taking settings from a valid legacy
version and applying to current OpenSearch Dashboards was made
here:

#485

However, it explicitly checked for current version being 1.0.0, which
is too strict because ideally all versions of 1.x is compatible.

This makes the config check more relaxed and will taking settings from
a legacy version if the current version is 1.x.

Issue resolved:
#741

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>
kavilla added a commit to kavilla/OpenSearch-Dashboards-1 that referenced this issue Aug 31, 2021
An original update to enable taking settings from a valid legacy
version and applying to current OpenSearch Dashboards was made
here:

opensearch-project#485

However, it explicitly checked for current version being 1.0.0, which
is too strict because ideally all versions of 1.x is compatible.

This makes the config check more relaxed and will taking settings from
a legacy version if the current version is 1.x.

Issue resolved:
opensearch-project#741

Backport PR:
opensearch-project#743

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>
kavilla added a commit that referenced this issue Aug 31, 2021
An original update to enable taking settings from a valid legacy
version and applying to current OpenSearch Dashboards was made
here:

#485

However, it explicitly checked for current version being 1.0.0, which
is too strict because ideally all versions of 1.x is compatible.

This makes the config check more relaxed and will taking settings from
a legacy version if the current version is 1.x.

Issue resolved:
#741

Backport PR:
#743

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>
kavilla added a commit to kavilla/OpenSearch-Dashboards-1 that referenced this issue Sep 1, 2021
An original update to enable taking settings from a valid legacy
version and applying to current OpenSearch Dashboards was made
here:

opensearch-project#485

However, it explicitly checked for current version being 1.0.0, which
is too strict because ideally all versions of 1.x is compatible.

This makes the config check more relaxed and will taking settings from
a legacy version if the current version is 1.x.

Issue resolved:
opensearch-project#741

Backport PR:
opensearch-project#743

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>
kavilla added a commit that referenced this issue Sep 1, 2021
An original update to enable taking settings from a valid legacy
version and applying to current OpenSearch Dashboards was made
here:

#485

However, it explicitly checked for current version being 1.0.0, which
is too strict because ideally all versions of 1.x is compatible.

This makes the config check more relaxed and will taking settings from
a legacy version if the current version is 1.x.

Issue resolved:
#741

Backport PR:
#743

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request v1.1.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants