Skip to content

Decision stream: move to chunked transfer by default#4413

Open
blotus wants to merge 3 commits intomasterfrom
lapi-decisions-stream-optimization
Open

Decision stream: move to chunked transfer by default#4413
blotus wants to merge 3 commits intomasterfrom
lapi-decisions-stream-optimization

Conversation

@blotus
Copy link
Copy Markdown
Member

@blotus blotus commented Apr 7, 2026

  • Always send the decisions list in chunk in the decisions stream.
  • Avoid some useless copies to intermediate structs
  • Use a single JSON encoder to encode each decision

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 7, 2026

@blotus: There are no 'kind' label on this PR. You need a 'kind' label to generate the release automatically.

  • /kind feature
  • /kind enhancement
  • /kind refactoring
  • /kind fix
  • /kind chore
  • /kind dependencies
Details

I am a bot created to help the crowdsecurity developers manage community feedback and contributions. You can check out my manifest file to understand my behavior and what I can do. If you want to use this for your project, you can check out the BirthdayResearch/oss-governance-bot repository.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 7, 2026

@blotus: There are no area labels on this PR. You can add as many areas as you see fit.

  • /area agent
  • /area local-api
  • /area cscli
  • /area appsec
  • /area security
  • /area configuration
Details

I am a bot created to help the crowdsecurity developers manage community feedback and contributions. You can check out my manifest file to understand my behavior and what I can do. If you want to use this for your project, you can check out the BirthdayResearch/oss-governance-bot repository.

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 7, 2026

Codecov Report

❌ Patch coverage is 86.36364% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 63.36%. Comparing base (fc1677b) to head (4c67090).

Files with missing lines Patch % Lines
pkg/apiserver/controllers/v1/decisions.go 81.63% 5 Missing and 4 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4413      +/-   ##
==========================================
+ Coverage   63.31%   63.36%   +0.04%     
==========================================
  Files         476      476              
  Lines       33796    33823      +27     
==========================================
+ Hits        21398    21431      +33     
+ Misses      10262    10227      -35     
- Partials     2136     2165      +29     
Flag Coverage Δ
bats 46.79% <86.36%> (+0.16%) ⬆️
unit-linux 36.08% <33.33%> (-0.11%) ⬇️
unit-windows 24.93% <0.00%> (+0.02%) ⬆️

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.

@blotus
Copy link
Copy Markdown
Member Author

blotus commented Apr 8, 2026

/kind enhancement

@github-actions github-actions bot added kind/enhancement New feature or request and removed needs/kind labels Apr 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/enhancement New feature or request needs/area

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant