Skip to content

m365_defender,sentinel_one_cloud_funnel: improve command line split script performance - #13335

Merged
efd6 merged 1 commit into
elastic:mainfrom
efd6:cmdline_perf-sentinel_one_cloud_funnel-m365_defender
Mar 30, 2025
Merged

m365_defender,sentinel_one_cloud_funnel: improve command line split script performance#13335
efd6 merged 1 commit into
elastic:mainfrom
efd6:cmdline_perf-sentinel_one_cloud_funnel-m365_defender

Conversation

@efd6

@efd6 efd6 commented Mar 27, 2025

Copy link
Copy Markdown
Contributor

Proposed commit message

m365_defender,sentinel_one_cloud_funnel: improve command line split script performance

Investigation in the crowdstrike fdr data stream has shown that this
processor can be significantly improved in terms of performance by
reducing string allocation while tokenising the command line. This
change replays the changes in that data stream in m365_defender and
sentinel_one_cloud_funnel.

ref: #13325

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Screenshots

@efd6 efd6 added enhancement New feature or request Integration:m365_defender Microsoft Defender XDR Integration:sentinel_one_cloud_funnel SentinelOne Cloud Funnel Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] labels Mar 27, 2025
@efd6 efd6 self-assigned this Mar 27, 2025
…cript performance

Investigation in the crowdstrike fdr data stream has shown that this
processor can be significantly improved in terms of performance by
reducing string allocation while tokenising the command line. This
change replays the changes in that data stream in m365_defender and
sentinel_one_cloud_funnel.

ref: elastic#13325
@efd6
efd6 force-pushed the cmdline_perf-sentinel_one_cloud_funnel-m365_defender branch from 33b4a88 to 6a5ec54 Compare March 27, 2025 21:48
@elastic-vault-github-plugin-prod

elastic-vault-github-plugin-prod Bot commented Mar 27, 2025

Copy link
Copy Markdown
Contributor

🚀 Benchmarks report

Package m365_defender 👍(3) 💚(0) 💔(0)

Expand to view
Data stream Previous EPS New EPS Diff (%) Result
alert 0 759.3 759.3 ( - %) 👍
event 0 612.37 612.37 ( - %) 👍
incident 0 893.66 893.66 ( - %) 👍

Package sentinel_one_cloud_funnel 👍(1) 💚(0) 💔(0)

Expand to view
Data stream Previous EPS New EPS Diff (%) Result
event 0 361.27 361.27 ( - %) 👍
@efd6
efd6 marked this pull request as ready for review March 27, 2025 22:23
@efd6
efd6 requested a review from a team as a code owner March 27, 2025 22:23
@elasticmachine

Copy link
Copy Markdown

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

@andrewkroh

Copy link
Copy Markdown
Contributor

/test benchmark fullreport

@elasticmachine

Copy link
Copy Markdown

💚 Build Succeeded

History

cc @efd6

@efd6
efd6 merged commit 16c5238 into elastic:main Mar 30, 2025
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

Package m365_defender - 3.1.0 containing this change is available at https://epr.elastic.co/package/m365_defender/3.1.0/

@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

Package sentinel_one_cloud_funnel - 1.11.0 containing this change is available at https://epr.elastic.co/package/sentinel_one_cloud_funnel/1.11.0/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request Integration:m365_defender Microsoft Defender XDR Integration:sentinel_one_cloud_funnel SentinelOne Cloud Funnel Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

4 participants