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

Processors #71

Closed
wants to merge 5 commits into from
Closed

Processors #71

wants to merge 5 commits into from

Conversation

corey-hammerton
Copy link
Contributor

Fixing YAML spacing discovered through filebeat configuration testing available in v0.9.0
Fixing README examples.

As described in the comments of templates/filebeat5.yml.erb and in
https://www.elastic.co/guide/en/beats/filebeat/current/configuration-processors.html,
we are providing puppet functionality to configure processors in filebeat.
The params values in the README.md lacked the required key for configuring
processor parameters. Instead of a params value being ["input_type", "offset"]
it must be {"fields" => ["input_type", "offset"]}.

Fixing the YAML spacing in templates/filebeat5.yml.erb thanks to the filebeat
config tester available in 0.9.0
…at into processors

Conflicts:
	README.md
	templates/filebeat5.yml.erb
@pcfens
Copy link
Owner

pcfens commented Jan 4, 2017

I merged 265fd17 from your fork using a cherry-pick to avoid having to mess with the README changes and all.

I've pushed it up to the forge as 0.9.1

Thanks

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

Successfully merging this pull request may close these issues.

None yet

2 participants