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

[netflow] Upgrade ECS to 8.0.0 #2424

Merged
merged 3 commits into from
Feb 3, 2022
Merged

Conversation

r00tu53r
Copy link
Contributor

@r00tu53r r00tu53r commented Jan 2, 2022

Automated PR.

Upgrades ECS to 8.0.0.

@elasticmachine
Copy link

Pinging @elastic/security-external-integrations (Team:Security-External Integrations)

@r00tu53r r00tu53r force-pushed the feat/ecs-8-0-netflow branch from 94171a3 to e2271cb Compare January 2, 2022 01:45
@r00tu53r r00tu53r mentioned this pull request Jan 2, 2022
@elasticmachine
Copy link

elasticmachine commented Jan 2, 2022

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2022-01-24T16:16:58.654+0000

  • Duration: 21 min 36 sec

  • Commit: a0e879a

Test stats 🧪

Test Results
Failed 0
Passed 92
Skipped 0
Total 92

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@r00tu53r r00tu53r requested review from marc-gr, leehinman and mtojek and removed request for marc-gr and leehinman January 13, 2022 03:34
@@ -563,7 +543,7 @@
- external: ecs
name: process.pid
- external: ecs
name: process.ppid
name: process.parent.pid
Copy link
Contributor

Choose a reason for hiding this comment

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

Breaking change?

Copy link
Contributor

Choose a reason for hiding this comment

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

Even though this field is defined, the Netflow package never populates fields under process, so the change doesn't have an impact.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants