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

Migrate: modules yaml and their json expected file to use the input and the input.type field #6121

Merged
merged 1 commit into from
Feb 3, 2018

Conversation

ph
Copy link
Contributor

@ph ph commented Jan 19, 2018

Migrate: modules yaml and their json expected file to use the input and input.type field
require: #6078

@ph ph added in progress Pull request is currently in progress. Filebeat Filebeat labels Jan 19, 2018
@ph ph mentioned this pull request Jan 19, 2018
10 tasks
@ph ph force-pushed the refactor/type-aliasing-input branch from d027f45 to 0e31dc7 Compare January 22, 2018 18:35
@ph ph force-pushed the refactor/rename-modules-to-use-input branch 2 times, most recently from c593ed0 to 2277291 Compare January 23, 2018 02:48
@ph ph changed the base branch from refactor/type-aliasing-input to master January 23, 2018 02:48
@ph ph added review and removed in progress Pull request is currently in progress. labels Jan 23, 2018
@ruflin
Copy link
Contributor

ruflin commented Jan 27, 2018

@ph I know I brought up this change but I have second thoughts now :-( Can we sync up next week?

@ph
Copy link
Contributor Author

ph commented Jan 27, 2018 via email

@ruflin
Copy link
Contributor

ruflin commented Jan 27, 2018

No, more if it's the correct field :-(

@ph
Copy link
Contributor Author

ph commented Jan 28, 2018 via email

@ph ph force-pushed the refactor/rename-modules-to-use-input branch 2 times, most recently from 319a1c5 to 668f088 Compare January 31, 2018 14:43
@ph ph changed the title Migrate: modules yaml and their json expected file to use the input and the new event.type from ECS Migrate: modules yaml and their json expected file to use the input and the input.type field Jan 31, 2018
@ph
Copy link
Contributor Author

ph commented Jan 31, 2018

jenkins test this please

@ph ph force-pushed the refactor/rename-modules-to-use-input branch from 668f088 to 6df5480 Compare February 2, 2018 15:45
@ruflin ruflin merged commit 70b2c99 into elastic:master Feb 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants