Skip to content

Add package with ingest pipelines in YAML#152

Merged
ruflin merged 1 commit intoelastic:masterfrom
ruflin:yaml-pipeline
Nov 14, 2019
Merged

Add package with ingest pipelines in YAML#152
ruflin merged 1 commit intoelastic:masterfrom
ruflin:yaml-pipeline

Conversation

@ruflin
Copy link
Copy Markdown
Collaborator

@ruflin ruflin commented Nov 14, 2019

This adds the yamlpipeline package with 3 ingest pipelines in yaml.

Closes #150

This adds the yamlpipeline package with 3 ingest pipelines in yaml.

Closes elastic#150
@ruflin ruflin requested a review from skh November 14, 2019 08:27
name: '{{IngestPipeline "pipeline-json" }}'
- pipeline:
if: ctx.message.charAt(0) != (char)("{")
name: '{{IngestPipeline "pipeline-plaintext" }}'
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

@skh We should still decide if we really want to use {{IngestPipeline as our identifier or should use something differently. In Beats we used {<.

Copy link
Copy Markdown
Contributor

@skh skh Nov 14, 2019

Choose a reason for hiding this comment

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

Then we should probably use {< too, because then the pipelines can be used from both projects. We could even accept both.

Copy link
Copy Markdown
Contributor

@skh skh left a comment

Choose a reason for hiding this comment

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

Thanks!

@ruflin ruflin merged commit 3b3d2d4 into elastic:master Nov 14, 2019
@ruflin ruflin deleted the yaml-pipeline branch November 14, 2019 10:47
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.

Add package with ingest pipelines in yaml format

2 participants