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

Dynamical ingest pipeline selection in ES output #2109

Merged
merged 1 commit into from
Aug 9, 2016

Conversation

urso
Copy link

@urso urso commented Jul 27, 2016

  • Add dynamic ingest pipeline selection based on outil.Select
  • Add api ingest node integration test
  • Update logstash integration tests
  • Add output plugin -> ingest node integration tests
  • Add default setting to full config

@urso urso added in progress Pull request is currently in progress. libbeat blocked labels Jul 27, 2016
@urso urso force-pushed the enh/2064-dynamic-es-pipeline branch from 7408c76 to 8697abf Compare July 27, 2016 19:56
@tsg
Copy link
Contributor

tsg commented Jul 28, 2016

Needs a rebase, I think. Otherwise, LGTM.

@urso urso force-pushed the enh/2064-dynamic-es-pipeline branch from 1ac9b43 to 9366c8e Compare July 28, 2016 12:29
@urso urso removed the blocked label Jul 28, 2016
@urso urso force-pushed the enh/2064-dynamic-es-pipeline branch 3 times, most recently from 2bbfdcb to ec3f3af Compare July 28, 2016 12:47
@urso urso added the blocked label Jul 28, 2016
@urso
Copy link
Author

urso commented Jul 28, 2016

blocked by #2118

@tsg
Copy link
Contributor

tsg commented Jul 29, 2016

Also needs docs & changelog. I'm OK to adding that to the meta ticket.

@urso urso force-pushed the enh/2064-dynamic-es-pipeline branch from ec3f3af to 1905f2b Compare July 29, 2016 13:49
@urso
Copy link
Author

urso commented Jul 29, 2016

Blocked by #2119

@urso urso force-pushed the enh/2064-dynamic-es-pipeline branch from ea8573c to 35beefb Compare August 3, 2016 11:45
@urso urso added review and removed blocked in progress Pull request is currently in progress. labels Aug 3, 2016
@urso
Copy link
Author

urso commented Aug 3, 2016

The docs will be updated in another PR. Docs depend on #2132
I wonder what to do with config.full.yml in libbeat.

- Add dynamic ingest pipeline selection based on outil.Select
- Add api ingest node integration test
- Update logstash integration tests
- Add output plugin -> ingest node integration tests
- Add default setting to full config
@urso urso force-pushed the enh/2064-dynamic-es-pipeline branch from 35beefb to 69443f7 Compare August 3, 2016 13:58
@tsg tsg merged commit 69e34e8 into elastic:master Aug 9, 2016
@urso urso deleted the enh/2064-dynamic-es-pipeline branch February 19, 2019 18:27
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