Skip to content

Comments

Minor streaming buffer cleanup/v2#10276

Closed
inashivb wants to merge 1 commit intoOISF:masterfrom
inashivb:streaming-buffer-cleanups/v2
Closed

Minor streaming buffer cleanup/v2#10276
inashivb wants to merge 1 commit intoOISF:masterfrom
inashivb:streaming-buffer-cleanups/v2

Conversation

@inashivb
Copy link
Member

Previous PR: #10161

Changes since v1:

  • drop the commit about node checks

StreamingBuffer is not required to find the intersecting regions, so,
don't pass it as a param to the fn.
@codecov
Copy link

codecov bot commented Jan 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (1dcf69b) 82.19% compared to head (2250d6c) 82.32%.
Report is 31 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #10276      +/-   ##
==========================================
+ Coverage   82.19%   82.32%   +0.13%     
==========================================
  Files         974      977       +3     
  Lines      271825   271950     +125     
==========================================
+ Hits       223416   223888     +472     
+ Misses      48409    48062     -347     
Flag Coverage Δ
fuzzcorpus 63.52% <100.00%> (+0.51%) ⬆️
suricata-verify 61.50% <40.00%> (+0.08%) ⬆️
unittests 62.81% <40.00%> (-0.04%) ⬇️

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

@suricata-qa
Copy link

Information:

ERROR: QA failed on SURI_TLPW2_autofp_suri_time.

field baseline test %
SURI_TLPW2_autofp_stats_chk
.uptime 182 193 106.04%

Pipeline 17844

@inashivb inashivb marked this pull request as ready for review January 30, 2024 08:12
@victorjulien victorjulien added this to the 8.0 milestone Jan 30, 2024
This was referenced Jan 30, 2024
@victorjulien
Copy link
Member

Merged in #10321, thanks!

@inashivb inashivb deleted the streaming-buffer-cleanups/v2 branch March 6, 2024 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants