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

[Filebeat] [Doc] Metricbeat host namespace causes mapping conflict when used with Logstash #7368

Closed
jkelastic opened this issue Jun 19, 2018 · 4 comments

Comments

@jkelastic
Copy link

jkelastic commented Jun 19, 2018

Need to add the following to the breaking changes part of the Filebeat / Logstash docs

Metricbeat host namespace causes mapping conflict when used with Logstash

For confirmed bugs, please report:

After upgrade to 6.3.0, some logsources will cause an error in logstash and will not make it to elasticsearch. 

The error: 
[2018-06-15T22:22:53,246][WARN ][logstash.outputs.elasticsearch] Could not index event to Elasticsearch. {:status=>400, :action=>["index", {:_id=>nil, :_index=>"logstash-2018.06.15", :_type=>"doc", :_routing=>nil}, #<LogStash::Event:0x5b4f0227>], :response=>{"index"=>{"_index"=>"logstash-2018.06.15", "_type"=>"doc", "_id"=>"rg0dBWQB0H0urKNSctIj", "status"=>400, "error"=>{"type"=>"mapper_parsing_exception", "reason"=>"failed to parse [host]", "caused_by"=>{"type"=>"illegal_state_exception", "reason"=>"Can't get text on a START_OBJECT at 1:620"}}}}}
@andrewkroh
Copy link
Member

To clarify, I think you are proposing to have it listed on a new "Breaking Changes in 6.3" page linked from here. Correct?

It is listed as a breaking change in the release notes.

@ppf2
Copy link
Member

ppf2 commented Jun 19, 2018

+1. Users generally go to breaking changes for breaking changes and may miss them in release notes.
For this particular one, I will suggest not only highlight it in breaking changes, but also provide a list of potential workarounds to address the field conflicting type issue.

@jkelastic
Copy link
Author

@andrewkroh yes, please also include it in the breaking changes as customer don't read the release notes.

@dedemorton dedemorton self-assigned this Jun 21, 2018
@dedemorton
Copy link
Contributor

Added a first draft of the breaking change docs here: #7398

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

4 participants