Skip to content

Add support for templates index_patternsfield in ES >= 6.0#4056

Merged
ruflin merged 1 commit intoelastic:masterfrom
exekias:support-index-patterns
Apr 20, 2017
Merged

Add support for templates index_patternsfield in ES >= 6.0#4056
ruflin merged 1 commit intoelastic:masterfrom
exekias:support-index-patterns

Conversation

@exekias
Copy link
Copy Markdown
Contributor

@exekias exekias commented Apr 19, 2017

The change was introduced in
elastic/elasticsearch#21009 and deprecates
old template field.

This change uses the new index_patterns field and keeps
compatibility with template for older version

closes #3133

Copy link
Copy Markdown
Contributor

@ruflin ruflin left a comment

Choose a reason for hiding this comment

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

LGTM. I suggest to add a changelog entry for this one.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice to have the link to the github issue here. makes it easy to track the change later again.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We should probably enhance version in the future to also support GreaterThenfunctions or similar. This will do for the moment.

The change was introduced in
elastic/elasticsearch#21009 and deprecates
old `template` field.

This change uses the new `index_patterns` field and keeps
compatibility with `template` for older version
@exekias exekias force-pushed the support-index-patterns branch from 3ede5e0 to 7589a16 Compare April 20, 2017 08:17
@exekias
Copy link
Copy Markdown
Contributor Author

exekias commented Apr 20, 2017

jenkins, retest this please

@ruflin ruflin merged commit 607c69b into elastic:master Apr 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Change template to index-patterns in index template

2 participants