Project

General

Profile

Actions
Copy link

Task #7461

open

suricata-verify: pass all tests

Added by Philippe Antoine 3 months ago. Updated 4 days ago.

Status:
In Review
Priority:
Normal
Assignee:
Philippe Antoine
Target version:
8.0.0-rc1
Effort:
Difficulty:
Label:

Related issues 4 (0 open — 4 closed)

Related to Suricata - Optimization #7416: suricata-verify: remove requires on file namesClosedOISF DevActions
Blocked by Suricata - Bug #7447: NULL dereference in ThreadLogFileHashFreeFunc in bug-5198 SV testClosedJeff LucovskyActions
Blocked by Suricata - Bug #7449: app-layer metadata does not get logged for stream rules and unidirectional protocolsClosedPhilippe AntoineActions
Blocked by Suricata - Bug #7462: suricata-verify: pass when compiled with FUZZING_UNSAFERejectedPhilippe AntoineActions
Actions
Copy link
 #1

Updated by Philippe Antoine 3 months ago

  • Blocked by Bug #7447: NULL dereference in ThreadLogFileHashFreeFunc in bug-5198 SV test added
Actions
Copy link
 #2

Updated by Philippe Antoine 3 months ago

Actions
Copy link
 #4

Updated by Philippe Antoine 3 months ago

  • Blocked by Bug #7449: app-layer metadata does not get logged for stream rules and unidirectional protocols added
Actions
Copy link
 #6

Updated by Philippe Antoine 3 months ago

Actions
Copy link
 #7

Updated by Philippe Antoine 3 months ago

Actions
Copy link
 #9

Updated by Philippe Antoine 3 months ago

Also
- DHCP test to enable
- Remove obsolete tests unified2 and http-headers

Actions
Copy link
 #10

Updated by Philippe Antoine 3 months ago

  • Blocked by Bug #7462: suricata-verify: pass when compiled with FUZZING_UNSAFE added
Actions
Copy link
 #11

Updated by Philippe Antoine 3 months ago

  • Status changed from New to In Review
Actions
Copy link
 #12

Updated by Victor Julien 3 months ago

  • Subject changed from pass all suricata-verify tests to suricata-verify: pass all tests
Actions
Copy link
 #13

Updated by Philippe Antoine about 1 month ago

I guess in the end we also need
- some CI check that we run all SV tests in some config
- check that version requires are not too much (by running with force option)

Actions
Copy link
 #14

Updated by Philippe Antoine 5 days ago

  • Target version changed from 8.0.0-beta1 to 8.0.0-rc1
Actions
Copy link
 #15

Updated by Philippe Antoine 5 days ago

@Jason Ish did you add a CI check with disable-ja3 ?

Actions
Copy link
 #16

Updated by Philippe Antoine 4 days ago

Running master with ./configure --enable-debug --enable-geoip I get the following SV tests that are not run

- versions check : https://github.com/OISF/suricata-verify/pull/2375 makes some more tests run on 8
- requires non-root user : bug-5198, filestore-v2.3-fserror, pcap-log-lz4-write : I checked them manually
- tls-ja3s-requires-off: SKIPPED: not for feature HAVE_JA3 and ja4-rules-requires-off: SKIPPED: not for feature HAVE_JA4, will be run by https://github.com/OISF/suricata/pull/12849
- feature UNITTESTS

Actions
Copy link

Also available in: Atom PDF