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

Document change to LS index setting in Breaking Changes #5614

Closed
andrewkroh opened this issue Nov 16, 2017 · 5 comments
Closed

Document change to LS index setting in Breaking Changes #5614

andrewkroh opened this issue Nov 16, 2017 · 5 comments
Assignees
Labels

Comments

@andrewkroh
Copy link
Member

Users that send data from Beats to LS need to update their LS configuration to specify a new index name that includes the Beat version. Previously we recommended

output {
  elasticsearch {
    hosts => "localhost:9200"
    manage_template => false
    index => "%{[@metadata][beat]}-%{+YYYY.MM.dd}" 
    document_type => "%{[@metadata][type]}" 
  }
}

And in 6.0 we updated to:

output {
  elasticsearch {
    hosts => "localhost:9200"
    manage_template => false
    index => "%{[@metadata][beat]}-%{[@metadata][version]}-%{+YYYY.MM.dd}" 
    document_type => "%{[@metadata][type]}" 
  }
}

The index templates we supply all apply to [beat]-[version]-* so if you don't update LS the template won't be applied.

@ruflin
Copy link
Contributor

ruflin commented Nov 27, 2017

@dedemorton Not sure if this was backported yet?

@dedemorton
Copy link
Contributor

@ruflin No, the PR has not been backported yet. However the topics that Kyle points to are about the Elasticsearch output config, not the Logstash configuration, which this issue focuses on.

I do think a small update of the ES output config docs is still required. We should show that the default index name is "filebeat-%{[beat.version]}-%{+yyyy.MM.dd}" correct?

But I'm not sure if we want to recommend that users include the beat version when they define a custom index name. WDYT?

@ruflin
Copy link
Contributor

ruflin commented Nov 29, 2017

We should recommend to include the beats version as otherwise in case we do bug fixes to the template and they update the version, they would not automatically get the newest template.

@dedemorton
Copy link
Contributor

Note that #5841 fixes the remaining examples to include the version info in the index name.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants