Skip to content

Conversation

@JeremyCloarec
Copy link
Contributor

@JeremyCloarec JeremyCloarec commented Oct 22, 2025

Proposed changes

  • add new thread pool dedicated to internal queues

linked to OpenCTI-Platform/client-python#1021

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 (coverage and e2e)
  • I added/update the relevant documentation (either on github or on notion)
  • Where necessary I refactored code to improve the overall quality

Further comments

@github-actions github-actions bot added the filigran team use to identify PR from the Filigran team label Oct 22, 2025
@JeremyCloarec JeremyCloarec linked an issue Oct 22, 2025 that may be closed by this pull request
@JeremyCloarec JeremyCloarec marked this pull request as ready for review October 22, 2025 14:58
@JeremyCloarec JeremyCloarec added the multi-repository For contribution that requires PR in several repository label Oct 23, 2025
@codecov
Copy link

codecov bot commented Oct 24, 2025

Codecov Report

❌ Patch coverage is 85.71429% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 16.32%. Comparing base (5cf1acc) to head (0dede55).
⚠️ Report is 18 commits behind head on master.

Files with missing lines Patch % Lines
...rm/opencti-graphql/src/modules/form/form-domain.ts 33.33% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master   #12896      +/-   ##
==========================================
+ Coverage   16.14%   16.32%   +0.17%     
==========================================
  Files        2821     2822       +1     
  Lines      409113   409414     +301     
  Branches    11263    11957     +694     
==========================================
+ Hits        66059    66832     +773     
+ Misses     343054   342582     -472     
Flag Coverage Δ
opencti 16.32% <85.71%> (+0.17%) ⬆️
opencti-front 3.92% <ø> (+0.03%) ⬆️
opencti-graphql 67.07% <85.71%> (+0.69%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.

"Primary motivation": "Motivation primaire",
"Print date": "Date d'impression",
"Priority": "Priorité",
"Priority group": "Groupe prioritaire",
Copy link
Member

Choose a reason for hiding this comment

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

i would have say 'Groupe de priorité' ?

@JeremyCloarec JeremyCloarec merged commit baefe5e into master Oct 24, 2025
11 checks passed
@JeremyCloarec JeremyCloarec deleted the issue/12490 branch October 24, 2025 13:07
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 multi-repository For contribution that requires PR in several repository

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Option to dedicate workers to specific queues

3 participants