-
Notifications
You must be signed in to change notification settings - Fork 507
crowdstrike: parse command line to populate process name in FDR logs #15646
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
crowdstrike: parse command line to populate process name in FDR logs #15646
Conversation
🚀 Benchmarks reportPackage
|
| Data stream | Previous EPS | New EPS | Diff (%) | Result |
|---|---|---|---|---|
falcon |
4405.29 | 3392.13 | -1013.16 (-23%) | 💔 |
host |
4566.21 | 2896.03 | -1670.18 (-36.58%) | 💔 |
fdr |
1459.85 | 1044.93 | -414.92 (-28.42%) | 💔 |
To see the full report comment with /test benchmark fullreport
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you, this looks great!
|
@navnit-elastic hey, just wanted to check in and see when is it planned to be released? :) Thank you! |
packages/crowdstrike/data_stream/fdr/elasticsearch/ingest_pipeline/default.yml
Outdated
Show resolved
Hide resolved
packages/crowdstrike/data_stream/fdr/elasticsearch/ingest_pipeline/default.yml
Show resolved
Hide resolved
packages/crowdstrike/data_stream/fdr/elasticsearch/ingest_pipeline/default.yml
Outdated
Show resolved
Hide resolved
Conflicts: packages/crowdstrike/changelog.yml packages/crowdstrike/manifest.yml
|
Pinging @elastic/security-service-integrations (Team:Security-Service Integrations) |
💚 Build Succeeded
History
|
|
Package crowdstrike - 2.6.0 containing this change is available at https://epr.elastic.co/package/crowdstrike/2.6.0/ |
…lastic#15646) This handles a special case occurs in Linux-based containerized environments when the "runc" process clones itself to get into its own namespace. The child process would have its executable path set to "/" which was resulting in "process.name" being empty. This change adds command line parsing to extract "process.name" when "process.executable" is set to a slash ("/"). Adds fields definition for ChangeTime, OciContainerId and RootPath.
Proposed commit message
Checklist
changelog.ymlfile.Author's Checklist
How to test this PR locally
Pipeline Tests:
Related issues
Screenshots