Skip to content

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

Merged
ruflin merged 1 commit intoelastic:masterfrom
ph:refactor/rename-modules-to-use-input
Feb 3, 2018
Merged

Migrate: modules yaml and their json expected file to use the input and the input.type field#6121
ruflin merged 1 commit intoelastic:masterfrom
ph:refactor/rename-modules-to-use-input

Conversation

@ph
Copy link
Copy Markdown
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 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
Copy Markdown
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
Copy Markdown
Contributor Author

ph commented Jan 27, 2018 via email

@ruflin
Copy link
Copy Markdown
Contributor

ruflin commented Jan 27, 2018

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

@ph
Copy link
Copy Markdown
Contributor Author

ph commented Jan 28, 2018 via email

@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
Copy Markdown
Contributor Author

ph commented Jan 31, 2018

jenkins test this please

@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