Skip to content

Use V2 templates when reading duplicate aliases and ingest pipelines#54902

Merged
dakrone merged 1 commit intoelastic:masterfrom
dakrone:itv2-check-v2-templates-everywhere
Apr 8, 2020
Merged

Use V2 templates when reading duplicate aliases and ingest pipelines#54902
dakrone merged 1 commit intoelastic:masterfrom
dakrone:itv2-check-v2-templates-everywhere

Conversation

@dakrone
Copy link
Copy Markdown
Member

@dakrone dakrone commented Apr 7, 2020

When a new index is rolled over, we check to see whether there are any duplicate alias
configurations in the index template configuration. Additionally, when a new index is created from a
bulk action, we check the templates to see if there are any ingest pipelines that need to be applied
to the index that will be newly created.

Both of these actions previously checked the v1 templates for their settings, they now also check
the v2 index templates, with the v2 index templates taking precendence similar to the way they do
when creating an index.

Relates to #53101

When a new index is rolled over, we check to see whether there are any duplicate alias
configurations in the index template configuration. Additionally, when a new index is created from a
bulk action, we check the templates to see if there are any ingest pipelines that need to be applied
to the index that will be newly created.

Both of these actions previously checked the v1 templates for their settings, they now also check
the v2 index templates, with the v2 index templates taking precendence similar to the way they do
when creating an index.

Relates to elastic#53101
@dakrone dakrone added :Data Management/Indices APIs DO NOT USE. Use ":Distributed/Indices APIs" or ":StorageEngine/Templates" instead. v8.0.0 v7.8.0 labels Apr 7, 2020
@dakrone dakrone requested review from martijnvg and probakowski April 7, 2020 15:01
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-core-features (:Core/Features/Indices APIs)

@dakrone
Copy link
Copy Markdown
Member Author

dakrone commented Apr 7, 2020

This is subsequent work as Martijn pointed out in #54669 (comment) and #54669 (comment)

@dakrone
Copy link
Copy Markdown
Member Author

dakrone commented Apr 7, 2020

@elasticmachine run elasticsearch-ci/bwc
@elasticmachine run elasticsearch-ci/packaging-sample-unix-docker

@dakrone dakrone requested a review from andreidan April 8, 2020 15:01
Copy link
Copy Markdown
Contributor

@andreidan andreidan left a comment

Choose a reason for hiding this comment

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

LGTM, thanks Lee

@dakrone dakrone merged commit e5d6fd4 into elastic:master Apr 8, 2020
@dakrone dakrone deleted the itv2-check-v2-templates-everywhere branch April 8, 2020 18:07
dakrone added a commit to dakrone/elasticsearch that referenced this pull request Apr 8, 2020
…tic#54902)

When a new index is rolled over, we check to see whether there are any duplicate alias
configurations in the index template configuration. Additionally, when a new index is created from a
bulk action, we check the templates to see if there are any ingest pipelines that need to be applied
to the index that will be newly created.

Both of these actions previously checked the v1 templates for their settings, they now also check
the v2 index templates, with the v2 index templates taking precendence similar to the way they do
when creating an index.

Relates to elastic#53101
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Data Management/Indices APIs DO NOT USE. Use ":Distributed/Indices APIs" or ":StorageEngine/Templates" instead. >enhancement v7.8.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants