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

Cherry-pick #4769 to 6.0: Allow template pattern to be overwritten #4965

Merged
merged 1 commit into from
Aug 22, 2017

Conversation

ruflin
Copy link
Contributor

@ruflin ruflin commented Aug 22, 2017

Cherry-pick of PR #4769 to 6.0 branch. Original message:

With the current template loading is was not possible to overload the pattern used as name and pattern were mixed. For example in the case where someone wanted to define pattern name a that applies to the indice a this was not possible because the pattern generated was a-* so it didn't apply to the index a. This change now allows to set pattern and name separately.

To make sure index pattern and templates are in sync, the beat will not start in case the index pattern was modified but the template patterns not. We should figure out later some automated mechanisms here.

We should find a better solution for 6.x to require less config options and also include dashboard generation in this.

With the current template loading is was not possible to overload the pattern used as name and pattern were mixed. For example in the case where someone wanted to define pattern name `a` that applies to the indice `a` this was not possible because the pattern generated was `a-*` so it didn't apply to the index `a`. This change now allows to set pattern and name separately.

To make sure index pattern and templates are in sync, the beat will not start in case the index pattern was modified but the template patterns not. We should figure out later some automated mechanisms here.

We should find a better solution for 6.x to require less config options and also include dashboard generation in this.

(cherry picked from commit 02a59a1)
@ruflin ruflin force-pushed the backport_4769_6.0 branch from 0d4e9ac to 32b4cb4 Compare August 22, 2017 07:25
@ruflin
Copy link
Contributor Author

ruflin commented Aug 22, 2017

jenkins, test it

@tsg tsg merged commit a9da916 into elastic:6.0 Aug 22, 2017
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
With the current template loading is was not possible to overload the pattern used as name and pattern were mixed. For example in the case where someone wanted to define pattern name `a` that applies to the indice `a` this was not possible because the pattern generated was `a-*` so it didn't apply to the index `a`. This change now allows to set pattern and name separately.

To make sure index pattern and templates are in sync, the beat will not start in case the index pattern was modified but the template patterns not. We should figure out later some automated mechanisms here.

We should find a better solution for 6.x to require less config options and also include dashboard generation in this.

(cherry picked from commit 2a8f795)
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.

2 participants