Skip to content

[backend] fix livestreamhandler monitoring fail on redis pipeline (#14604)#14621

Merged
xfournet merged 1 commit intomasterfrom
issue/14604
Feb 26, 2026
Merged

[backend] fix livestreamhandler monitoring fail on redis pipeline (#14604)#14621
xfournet merged 1 commit intomasterfrom
issue/14604

Conversation

@xfournet
Copy link
Member

@xfournet xfournet commented Feb 25, 2026

Proposed changes

  • update the slot prefix to execute pipeline on a single slot

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

@codecov
Copy link

codecov bot commented Feb 25, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 32.36%. Comparing base (0a6e615) to head (897aa87).
⚠️ Report is 6 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #14621   +/-   ##
=======================================
  Coverage   32.36%   32.36%           
=======================================
  Files        3096     3096           
  Lines      210896   210896           
  Branches    38197    38197           
=======================================
  Hits        68262    68262           
  Misses     142634   142634           
Flag Coverage Δ
opencti-client-python 45.50% <ø> (ø)
opencti-front 2.83% <ø> (ø)
opencti-graphql 67.74% <100.00%> (ø)

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.

SamuelHassine added a commit that referenced this pull request Feb 25, 2026
The AIInsights component was accidentally removed from the Report Root.tsx
during the v7 UI overhaul. Additionally, enableAskAi was explicitly set to
false on the ContainerHeader.

This commit:
- Re-adds the AIInsights import
- Renders AIInsights alongside StixCoreObjectSecurityCoverage in the tab bar
- Removes the explicit enableAskAi={false} from ContainerHeader

Mirrors the existing working implementation in Groupings Root.tsx.

Closes #14621
@xfournet xfournet self-assigned this Feb 26, 2026
@xfournet xfournet added the filigran team use to identify PR from the Filigran team label Feb 26, 2026
@xfournet xfournet merged commit 81c5215 into master Feb 26, 2026
54 of 56 checks passed
@xfournet xfournet deleted the issue/14604 branch February 26, 2026 11:10
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.

Stream connector issue when running Redis as a cluster

2 participants