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

Throw error when migration not defined in same plugin as mappings #31739

Conversation

mikecote
Copy link
Contributor

@mikecote mikecote commented Feb 21, 2019

Fixes #31694.

This PR makes the system throw an error when a plugin defines a migration but doesn't define/own the mappings. This is to enforce migrations to be defined in the same plugin that defines the mappings.

Dev Docs

Kibana will now throw an error on startup if a plugin has defined a saved object migration inside a plugin that doesn't define or own the mappings to.

@mikecote mikecote added Team:Operations Team label for Operations Team v8.0.0 v7.2.0 labels Feb 21, 2019
@mikecote mikecote self-assigned this Feb 21, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-operations

@elasticmachine

This comment has been minimized.

@mikecote mikecote requested review from spalger February 26, 2019 15:40
@elasticmachine

This comment has been minimized.

Copy link
Contributor

@spalger spalger left a comment

Choose a reason for hiding this comment

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

LGTM

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@mikecote mikecote merged commit d4488d6 into elastic:master Mar 4, 2019
mikecote added a commit to mikecote/kibana that referenced this pull request Mar 4, 2019
…astic#31739)

* Throw error when migration not defined in same plugin as mappings

* Apply PR feedback
mikecote added a commit that referenced this pull request Mar 4, 2019
…1739) (#32411)

* Throw error when migration not defined in same plugin as mappings

* Apply PR feedback
@gchaps
Copy link
Contributor

gchaps commented Jun 12, 2019

@mikecote Please add some information about this change to the 7.2 breaking changes doc at https://www.elastic.co/guide/en/kibana/7.2/breaking-changes-7.2.html

mikecote added a commit to mikecote/kibana that referenced this pull request Jun 12, 2019
mikecote added a commit to mikecote/kibana that referenced this pull request Jun 12, 2019
mikecote added a commit that referenced this pull request Jun 12, 2019
mikecote added a commit that referenced this pull request Jun 12, 2019
mikecote added a commit that referenced this pull request Jun 12, 2019
mikecote added a commit that referenced this pull request Jun 12, 2019
@mikecote mikecote added release_note:plugin_api_changes Contains a Plugin API changes section for the breaking plugin API changes section. and removed release_note:breaking labels Jun 12, 2019
mikecote added a commit that referenced this pull request Jun 12, 2019
mikecote added a commit that referenced this pull request Jun 12, 2019
@elasticmachine
Copy link
Contributor

💔 Build Failed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:plugin_api_changes Contains a Plugin API changes section for the breaking plugin API changes section. review Team:Operations Team label for Operations Team v7.2.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enforce migrations to be defined in the same plugin as the mappings
5 participants