Skip to content

[core/server/plugins] don't run discovery in dev server parent process#79235

Merged
spalger merged 3 commits intoelastic:masterfrom
spalger:fix/prevent-unnecessary-plugin-discovery
Oct 2, 2020
Merged

[core/server/plugins] don't run discovery in dev server parent process#79235
spalger merged 3 commits intoelastic:masterfrom
spalger:fix/prevent-unnecessary-plugin-discovery

Conversation

@spalger
Copy link
Copy Markdown
Contributor

@spalger spalger commented Oct 1, 2020

Extracted from #78710

This prevents the dev server from loading plugins in the core when it's started just for the purpose of launching the ClusterManager and BasePathProxy.

@spalger spalger added Team:Core Platform Core services: plugins, logging, config, saved objects, http, ES client, i18n, etc t// v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.10.0 labels Oct 1, 2020
@spalger spalger requested a review from a team as a code owner October 1, 2020 23:24
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-platform (Team:Platform)

@spalger
Copy link
Copy Markdown
Contributor Author

spalger commented Oct 2, 2020

@elasticmachine merge upstream

Copy link
Copy Markdown
Contributor

@pgayvallet pgayvallet left a comment

Choose a reason for hiding this comment

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

Can we just add a unit test, for when we'll decide of a more heavy refactoring of the behavior when isDevClusterMaster is true?

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

@spalger spalger merged commit 9021c83 into elastic:master Oct 2, 2020
@spalger spalger deleted the fix/prevent-unnecessary-plugin-discovery branch October 2, 2020 19:06
spalger added a commit to spalger/kibana that referenced this pull request Oct 2, 2020
elastic#79235)

Co-authored-by: spalger <spalger@users.noreply.github.com>
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
spalger added a commit that referenced this pull request Oct 2, 2020
spalger added a commit to spalger/kibana that referenced this pull request Oct 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:skip Skip the PR/issue when compiling release notes reverted Team:Core Platform Core services: plugins, logging, config, saved objects, http, ES client, i18n, etc t// v7.10.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants