Skip to content

Conversation

@damgouj
Copy link
Member

@damgouj damgouj commented Nov 26, 2025

Proposed changes

  • Add SentinelOne executor

Testing Instructions

  1. Enable SentinelOne Executor (see the notion page and the doc here [docs] SentinelOne executor docs#232)
  2. Enable EE
  3. Check if the doc is clear
  4. Check if asset groups and assets are created for your SentinelOne agents
  5. Check if you can execute payloads with your SentinelOne agents (Windows and Unix)

Related issues

Checklist

  • I consider the submitted work as finished
  • I tested the code for its functionality
  • I wrote test cases for the relevant uses case
  • I added/update the relevant documentation (either on github or on notion)
  • Where necessary I refactored code to improve the overall quality
  • For bug fix -> I implemented a test that covers the bug

Further comments

Don't hesitate if necessary !

@damgouj damgouj added this to the Release 2.1.0 milestone Nov 26, 2025
@damgouj damgouj self-assigned this Nov 26, 2025
@damgouj damgouj added the filigran team use to identify PR from the Filigran team label Nov 26, 2025
@damgouj damgouj changed the title [backend/frontend] feat(executors): Add SentinelOne executor - chunk 1 (#3515) [backend/frontend] feat(executors): add SentinelOne executor - chunk 1 (#3515) Nov 26, 2025
@codecov
Copy link

codecov bot commented Nov 26, 2025

Codecov Report

❌ Patch coverage is 46.17284% with 218 lines in your changes missing coverage. Please review.
✅ Project coverage is 50.13%. Comparing base (3905cff) to head (6ab6cb7).
⚠️ Report is 1 commits behind head on release/current.

Files with missing lines Patch % Lines
.../sentinelone/client/SentinelOneExecutorClient.java 1.38% 71 Missing ⚠️
...one/service/SentinelOneExecutorContextService.java 65.21% 36 Missing and 4 partials ⚠️
...entinelone/service/SentinelOneExecutorService.java 76.63% 18 Missing and 7 partials ⚠️
...ne/service/SentinelOneGarbageCollectorService.java 20.00% 24 Missing ⚠️
...ain/java/io/openaev/executors/ExecutorService.java 0.00% 18 Missing ⚠️
...io/openaev/execution/ExecutionExecutorService.java 23.07% 9 Missing and 1 partial ⚠️
...src/main/java/io/openaev/service/AgentService.java 0.00% 7 Missing ⚠️
...src/main/java/io/openaev/service/AssetService.java 0.00% 5 Missing ⚠️
...ke/service/CrowdStrikeGarbageCollectorService.java 0.00% 4 Missing ⚠️
...utors/sentinelone/SentinelOneGarbageCollector.java 0.00% 4 Missing ⚠️
... and 4 more
Additional details and impacted files
@@                  Coverage Diff                  @@
##             release/current    #4439      +/-   ##
=====================================================
- Coverage              50.19%   50.13%   -0.06%     
- Complexity              3659     3684      +25     
=====================================================
  Files                    900      907       +7     
  Lines                  26678    27041     +363     
  Branches                1998     2027      +29     
=====================================================
+ Hits                   13390    13556     +166     
- Misses                 12482    12666     +184     
- Partials                 806      819      +13     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@guillaumejparis guillaumejparis self-requested a review December 1, 2025 14:57
@guillaumejparis guillaumejparis removed their request for review December 2, 2025 13:44
@damgouj damgouj requested a review from Dimfacion December 3, 2025 10:06
@damgouj damgouj merged commit e11b773 into release/current Dec 3, 2025
11 checks passed
@damgouj damgouj deleted the issue/3515 branch December 3, 2025 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

filigran team use to identify PR from the Filigran team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants