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

[Serverless] Disable Migrate plugin #157881

Conversation

ElenaStoeva
Copy link
Contributor

@ElenaStoeva ElenaStoeva commented May 16, 2023

Summary

This PR makes the Migrate plugin disable-able for serverless.

Partially addresses #157756

How to test:

  1. Start Elasticsearch with yarn es snapshot and Kibana with yarn serverless-{mode} where {mode} can be es, security, or oblt.
  2. Verify that the Migrate plugin doesn't show up in the nav bar and its path (management/data/migrate_data) leads to the Stack Management landing page.

@ElenaStoeva ElenaStoeva added the Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more label May 16, 2023
@ElenaStoeva ElenaStoeva requested a review from a team as a code owner May 16, 2023 14:11
@ElenaStoeva ElenaStoeva self-assigned this May 16, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/platform-deployment-management (Team:Deployment Management)

@ElenaStoeva ElenaStoeva added the release_note:skip Skip the PR/issue when compiling release notes label May 16, 2023
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Unknown metric groups

ESLint disabled line counts

id before after diff
enterpriseSearch 19 21 +2
securitySolution 400 404 +4
total +6

Total ESLint disabled count

id before after diff
enterpriseSearch 20 22 +2
securitySolution 480 484 +4
total +6

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

cc @ElenaStoeva

Copy link
Contributor

@claracruz claracruz left a comment

Choose a reason for hiding this comment

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

Pulled and tested 🎉

@ElenaStoeva
Copy link
Contributor Author

Thanks for the review @claracruz!

On another note, are you aware of any other plugins/places in Kibana that have links to the Migrate plugin? I didn't find any dependencies, but wanted to check with you to make sure.

@ElenaStoeva ElenaStoeva merged commit 574d334 into elastic:main May 18, 2023
@kibanamachine kibanamachine added v8.9.0 backport:skip This commit does not require backporting labels May 18, 2023
@alisonelizabeth
Copy link
Contributor

On another note, are you aware of any other plugins/places in Kibana that have links to the Migrate plugin? I didn't find any dependencies, but wanted to check with you to make sure.

There shouldn't be any dependencies. Thanks for checking!

@ElenaStoeva ElenaStoeva deleted the serverless/stack-management/disable-migrate branch May 22, 2023 08:50
delanni pushed a commit to delanni/kibana that referenced this pull request May 25, 2023
## Summary

This PR makes the Migrate plugin disable-able for serverless.

Partially addresses elastic#157756

**How to test:**

1. Start Elasticsearch with `yarn es snapshot` and Kibana with yarn
`serverless-{mode}` where `{mode}` can be `es`, `security`, or `oblt`.
2. Verify that the Migrate plugin doesn't show up in the nav bar and its
path (`management/data/migrate_data`) leads to the Stack Management
landing page.

---------

Co-authored-by: kibanamachine <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more v8.9.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants