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

[home/sampleData] add empty migrationVersion to auto-migration objects #29226

Merged

Conversation

spalger
Copy link
Contributor

@spalger spalger commented Jan 23, 2019

The sample data does not currently specify a migration version, which means we have manually update the data sets when updating the saved objects API. This just sets the migration version for all the saved objects to {}, which signals to the saved objects API that the object has not been migrated and should have all migrations applied.

@spalger spalger added v7.0.0 Feature:Home Kibana home application Team:Visualizations Visualization editors, elastic-charts and infrastructure v6.7.0 labels Jan 23, 2019
@spalger spalger requested a review from nreese January 23, 2019 23:37
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app

@nreese
Copy link
Contributor

nreese commented Jan 23, 2019

What about Sample data saved objects from Canvas and GIS? Should those be updated as well?

https://github.com/elastic/kibana/blob/master/x-pack/plugins/canvas/init.js#L10

https://github.com/elastic/kibana/blob/master/x-pack/plugins/gis/index.js#L63

@spalger spalger requested a review from a team as a code owner January 24, 2019 00:00
Copy link
Contributor

@nreese nreese left a comment

Choose a reason for hiding this comment

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

lgtm
code review, tested by loaded sample data saved objects and cycling kibana server to run migrations. No errors in kibana log.

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@spalger spalger merged commit 9bdee51 into elastic:master Jan 24, 2019
spalger pushed a commit to spalger/kibana that referenced this pull request Jan 24, 2019
elastic#29226)

The sample data does not currently specify a migration version, which means we have manually update the data sets when updating the saved objects API. This just sets the migration version for all the saved objects to `{}`, which signals to the saved objects API that the object has not been migrated and should have all migrations applied.
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

spalger pushed a commit that referenced this pull request Jan 24, 2019
#29226) (#29230)

The sample data does not currently specify a migration version, which means we have manually update the data sets when updating the saved objects API. This just sets the migration version for all the saved objects to `{}`, which signals to the saved objects API that the object has not been migrated and should have all migrations applied.
@spalger
Copy link
Contributor Author

spalger commented Jan 24, 2019

6.x/6.7: 173dadc

@spalger spalger deleted the fix/add-migration-versions-to-sample-data branch January 24, 2019 18:01
@w33ble
Copy link
Contributor

w33ble commented Jan 24, 2019

FWIW, I did try this in Canvas, and all the sample workpads worked fine. I also tried it with an existing instance, and everything worked as expected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Home Kibana home application Team:Visualizations Visualization editors, elastic-charts and infrastructure v6.7.0 v7.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants