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

Add event.ingested to all rsa2elk modules #20714

Merged
merged 1 commit into from
Aug 24, 2020

Conversation

adriansr
Copy link
Contributor

@adriansr adriansr commented Aug 20, 2020

Updated the autogenerated ingest pipelines to add the event.ingested field.

Most pipelines already had the processor due to #20386, just a few of them were missing. I think because this pipelines don't have tests.

Updated the autogenerated ingest pipelines to add the event.ingested
field.

Most pipelines already had the processor, just a few of them were
missing.
@elasticmachine
Copy link
Collaborator

Pinging @elastic/siem (Team:SIEM)

@botelastic botelastic bot added needs_team Indicates that the issue/PR needs a Team:* label and removed needs_team Indicates that the issue/PR needs a Team:* label labels Aug 20, 2020
Copy link
Member

@andrewkroh andrewkroh left a comment

Choose a reason for hiding this comment

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

LGTM.

I was wondering question how I missed the one since I added a test on the generated data, but then I saw "I think because this pipelines don't have tests."

@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: [Pull request #20714 opened]

  • Start Time: 2020-08-20T17:12:16.732+0000

  • Duration: 54 min 10 sec

Test stats 🧪

Test Results
Failed 0
Passed 2474
Skipped 388
Total 2862

@adriansr adriansr merged commit 6636a8c into elastic:master Aug 24, 2020
adriansr added a commit to adriansr/beats that referenced this pull request Aug 24, 2020
Updated the autogenerated ingest pipelines to add the event.ingested
field.

Most pipelines already had the processor, just a few of them were
missing.

(cherry picked from commit 6636a8c)
adriansr added a commit that referenced this pull request Aug 24, 2020
Updated the autogenerated ingest pipelines to add the event.ingested
field.

Most pipelines already had the processor, just a few of them were
missing.

(cherry picked from commit 6636a8c)
melchiormoulin pushed a commit to melchiormoulin/beats that referenced this pull request Oct 14, 2020
Updated the autogenerated ingest pipelines to add the event.ingested
field.

Most pipelines already had the processor, just a few of them were
missing.
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.

3 participants