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

Beats should not enforce setup.template.name and pattern if templates are disabled. #5308

Closed
pmusa opened this issue Oct 3, 2017 · 2 comments

Comments

@pmusa
Copy link

pmusa commented Oct 3, 2017

When you set output.elasticsearch.index: "my_index" Beats force you to also set setup.template.name and setup.template.pattern (Exiting: setup.template.name and setup.template.pattern have to be set if index name is modified.). However, if you don't want Beats to manage templates and set setup.template.enabled: false it still complains about the settings above, but it should not.

@kurtado
Copy link
Contributor

kurtado commented Oct 3, 2017

Maybe these could be added to the default config and just left commented out? Would be much easier for users to figure out and fix.

ruflin added a commit to ruflin/beats that referenced this issue Oct 4, 2017
If the elasticsearch index name is changed, its required to set a template. The error message was also shown when template itself are disabled. This is now changed that if templates are disabled, no error is returned.

Closes elastic#5308
@ruflin
Copy link
Contributor

ruflin commented Oct 4, 2017

I opened a PR here to fix the issue: #5319

@kurtado If this will be a common issue that people change the index pattern, we should add it to the default config options to make it easily modifiable. I'm kind of hoping the default is not changed that often as also the setting to change the ES index is not in the default config.

@tsg tsg closed this as completed in #5319 Oct 5, 2017
tsg pushed a commit that referenced this issue Oct 5, 2017
* Do not require template if index change and template disabled

If the elasticsearch index name is changed, its required to set a template. The error message was also shown when template itself are disabled. This is now changed that if templates are disabled, no error is returned.

Closes #5308

* Add Changelog
ruflin added a commit to ruflin/beats that referenced this issue Oct 5, 2017
…c#5319)

* Do not require template if index change and template disabled

If the elasticsearch index name is changed, its required to set a template. The error message was also shown when template itself are disabled. This is now changed that if templates are disabled, no error is returned.

Closes elastic#5308

* Add Changelog

(cherry picked from commit ce9e47b)
tsg pushed a commit that referenced this issue Oct 6, 2017
…#5334)

* Do not require template if index change and template disabled

If the elasticsearch index name is changed, its required to set a template. The error message was also shown when template itself are disabled. This is now changed that if templates are disabled, no error is returned.

Closes #5308

* Add Changelog

(cherry picked from commit ce9e47b)
leweafan pushed a commit to leweafan/beats that referenced this issue Apr 28, 2023
…c#5319) (elastic#5334)

* Do not require template if index change and template disabled

If the elasticsearch index name is changed, its required to set a template. The error message was also shown when template itself are disabled. This is now changed that if templates are disabled, no error is returned.

Closes elastic#5308

* Add Changelog

(cherry picked from commit b56accf)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants