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

[Index Management] Adopt data stream API changes #71682

Conversation

alisonelizabeth
Copy link
Contributor

@alisonelizabeth alisonelizabeth commented Jul 14, 2020

Multi-part towards #71670

This PR updates the data streams API integration test and fixes the composable templates creation flow.

How to test

Verify the composable index template creation flow works as expected when enabling the data streams toggle.

@alisonelizabeth alisonelizabeth added release_note:fix Feature:Index Management Index and index templates UI v8.0.0 Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more v7.9.0 labels Jul 14, 2020
@alisonelizabeth alisonelizabeth requested a review from a team as a code owner July 14, 2020 15:59
@elasticmachine
Copy link
Contributor

Pinging @elastic/es-ui (Team:Elasticsearch UI)

@alisonelizabeth
Copy link
Contributor Author

@tylersmalley that code is part of Snapshot and Restore. I believe @jloleysens said no changes are needed there, but going to look into again and verify.

@alisonelizabeth
Copy link
Contributor Author

@tylersmalley Update - I believe the Snapshot + Restore code you pointed to is still valid and no changes are needed. The ES change only relates to when defining a data stream while creating a composable index template.

Copy link
Contributor

@cjcenizal cjcenizal left a comment

Choose a reason for hiding this comment

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

Tested with unverified snapshot, was able to create a data stream index template. Code LGTM. Thanks for this fast fix, Alison!

return {
timestamp_field: '@timestamp',
};
// For now, ES expects an empty empty when defining a data stream
Copy link
Contributor

Choose a reason for hiding this comment

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

Typo: "empty empty" -> "empty object"

@alisonelizabeth
Copy link
Contributor Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Build metrics

✅ unchanged

History

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

@tylersmalley tylersmalley merged commit 919e0f6 into elastic:master Jul 14, 2020
tylersmalley pushed a commit to tylersmalley/kibana that referenced this pull request Jul 14, 2020
tylersmalley pushed a commit that referenced this pull request Jul 14, 2020
Co-authored-by: Alison Goryachev <alison.goryachev@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Index Management Index and index templates UI release_note:fix Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more v7.9.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants