Skip to content

[worker] add new thread pool dedicated to internal queues (#12490) - #12896

Merged
Jeremy Cloarec (JeremyCloarec) merged 7 commits into
masterfrom
issue/12490
Oct 24, 2025
Merged

Jeremy Cloarec (JeremyCloarec) merged 7 commits into
masterfrom
issue/12490

Conversation

@JeremyCloarec

@JeremyCloarec Jeremy Cloarec (JeremyCloarec) commented Oct 22, 2025 •

Copy link
Copy Markdown
Contributor

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 Item from the Filigran team. label Oct 22, 2025
@JeremyCloarec Jeremy Cloarec (JeremyCloarec) linked an issue Oct 22, 2025 that may be closed by this pull request
@JeremyCloarec
Jeremy Cloarec (JeremyCloarec) marked this pull request as ready for review October 22, 2025 14:58
@JeremyCloarec Jeremy Cloarec (JeremyCloarec) added the multi-repository For contribution that requires PR in several repository label Oct 23, 2025
@codecov

codecov Bot commented Oct 24, 2025 •

Copy link
Copy Markdown

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
Copy Markdown
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
Jeremy Cloarec (JeremyCloarec) merged commit baefe5e into master Oct 24, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

filigran team Item 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

2 participants