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

[Enterprise Search] Error on version mismatches #119101

Merged

Conversation

JasonStoltz
Copy link
Member

@JasonStoltz JasonStoltz commented Nov 18, 2021

Summary

This PR changes the Enterprise Search plugin to show an error message if the Kibana and Enterprise Search major and minor versions do not match.

error

When testing this, be sure to test entering the plugin via the Enterprise Search, App Search, and Workplace Search links. It's also worthwhile to try deep linking into the product as well. In all cases, a user should be presented the error message and prevented from using the plugin.

Checklist

Delete any items that are not applicable to this PR.

For maintainers

@JasonStoltz JasonStoltz force-pushed the kibana-ent-search-version-checks branch from bd93957 to 6dfb918 Compare November 19, 2021 13:45
@JasonStoltz JasonStoltz force-pushed the kibana-ent-search-version-checks branch from 6dfb918 to 54fd1ae Compare November 19, 2021 13:50
@JasonStoltz JasonStoltz marked this pull request as ready for review November 19, 2021 13:51
@JasonStoltz JasonStoltz requested review from a team November 19, 2021 13:51
@JasonStoltz JasonStoltz added v8.0.0 auto-backport Deprecated - use backport:version if exact versions are needed v8.1.0 release_note:skip Skip the PR/issue when compiling release notes labels Nov 19, 2021
Copy link
Contributor

@richkuz richkuz left a comment

Choose a reason for hiding this comment

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

The logic looks sound to me. Worth having someone in Workplace Search take a glance, too.

We should flag this for light manual testing once the 8.0 BC is available.

@@ -68,6 +68,8 @@ export const callEnterpriseSearchConfigAPI = async ({
warnMismatchedVersions(data?.version?.number, log);

return {
enterpriseSearchVersion: data?.version?.number,
kibanaVersion: kibanaPackageJson.version,
Copy link
Contributor

Choose a reason for hiding this comment

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

Minor optional nit, feel free to ignore: warnMismatchedVersions above could be modified to take a kibanaVersion, so it doesn't have to know to look at kibanaPackageJson.version itself.

@JasonStoltz
Copy link
Member Author

Thanks @richkuz. Just pushed up a couple of commits to resolve CI issues. I also noticed that our warning message was using different logic, that actually warned on patch differences as well, so I updated that to use the shared version check: 8cfd193.

Copy link
Contributor

@scottybollinger scottybollinger left a comment

Choose a reason for hiding this comment

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

Workplace Search changes LGTM

@JasonStoltz
Copy link
Member Author

@elasticmachine merge upstream

@JasonStoltz JasonStoltz enabled auto-merge (squash) November 22, 2021 14:37
@JasonStoltz JasonStoltz merged commit 7546174 into elastic:main Nov 22, 2021
@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Test Failures

  • [job] [logs] Default CI Group #1 / Stack Management registers all UI Settings in the UsageStats interface

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
enterpriseSearch 1320 1324 +4

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
enterpriseSearch 1.3MB 1.3MB +1.8KB

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Backport successful

Status Branch Result
8.0

This backport PR will be merged automatically after passing CI.

@JasonStoltz JasonStoltz deleted the kibana-ent-search-version-checks branch November 22, 2021 16:16
kibanamachine added a commit that referenced this pull request Nov 22, 2021
Co-authored-by: Jason Stoltzfus <jastoltz24@gmail.com>
TinLe pushed a commit to TinLe/kibana that referenced this pull request Dec 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed release_note:skip Skip the PR/issue when compiling release notes v8.0.0 v8.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants