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 #14010 to 7.x: [Filebeat] Select output index based on the source input #15094

Merged
merged 3 commits into from
Dec 13, 2019

Conversation

faec
Copy link
Contributor

@faec faec commented Dec 13, 2019

Cherry-pick of PR #14010 to 7.x branch. Original message:

Adds configuration to Filebeat inputs to override the output index of their events (#13255)

@ycombinator
Copy link
Contributor

Looks like the intake CI job failed with:

# github.com/elastic/beats/filebeat/channel
vet: channel/connector_test.go:87:14: too few arguments in call to actions.NewAddFields

@faec
Copy link
Contributor Author

faec commented Dec 13, 2019

Yeah, a change to one of the unit test's processor dependencies was backported before this was. Fixed it to match the current version in the main branch.

@@ -437,6 +437,8 @@ https://github.com/elastic/beats/compare/v7.0.0-alpha2...master[Check the HEAD d
- Fix timezone parsing in haproxy pipeline. {pull}14755[14755]
- Add module for ActiveMQ. {pull}14840[14840]
- Add dashboards for the ActiveMQ Filebeat module. {pull}14880[14880]
- Add `index` option to all inputs to directly set a per-input index value. {pull}14010[14010]
- Remove beta flag for some filebeat modules. {pull}14374[14374]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove this line?

Copy link
Contributor

@ycombinator ycombinator left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. WFG.

@faec
Copy link
Contributor Author

faec commented Dec 13, 2019

Only remaining failure is unrelated (in x-pack/metricbeat which has been flaky today), merging!

@faec faec merged commit 1bbac9f into elastic:7.x Dec 13, 2019
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