Task #7461
closedsuricata-verify: pass all tests
Added by Philippe Antoine over 1 year ago. Updated 9 months ago.
PA Updated by Philippe Antoine over 1 year ago Actions #1
- Blocked by Bug #7447: NULL dereference in ThreadLogFileHashFreeFunc in bug-5198 SV test added
PA Updated by Philippe Antoine over 1 year ago Actions #2
- Blocks Optimization #7416: suricata-verify: remove requires on file names added
PA Updated by Philippe Antoine over 1 year ago Actions #3
- Blocks Security #7450: tracking: signature can allocate arbitrary amount of memory added
PA Updated by Philippe Antoine over 1 year ago Actions #4
- Blocked by Bug #7449: app-layer metadata does not get logged for stream rules and unidirectional protocols added
PA Updated by Philippe Antoine over 1 year ago Actions #5
- Blocks deleted (Security #7450: tracking: signature can allocate arbitrary amount of memory)
PA Updated by Philippe Antoine over 1 year ago Actions #6
- Blocks deleted (Optimization #7416: suricata-verify: remove requires on file names)
PA Updated by Philippe Antoine over 1 year ago Actions #7
- Related to Optimization #7416: suricata-verify: remove requires on file names added
PA Updated by Philippe Antoine over 1 year ago Actions #8
- Related to Security #7450: tracking: signature can allocate arbitrary amount of memory added
PA Updated by Philippe Antoine over 1 year ago Actions #9
Also
- DHCP test to enable
- Remove obsolete tests unified2 and http-headers
PA Updated by Philippe Antoine over 1 year ago Actions #10
- Blocked by Bug #7462: suricata-verify: pass when compiled with FUZZING_UNSAFE added
PA Updated by Philippe Antoine over 1 year ago Actions #11
- Status changed from New to In Review
VJ Updated by Victor Julien over 1 year ago Actions #12
- Subject changed from pass all suricata-verify tests to suricata-verify: pass all tests
PA Updated by Philippe Antoine about 1 year ago Actions #13
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)
PA Updated by Philippe Antoine about 1 year ago Actions #14
- Target version changed from 8.0.0-beta1 to 8.0.0-rc1
PA Updated by Philippe Antoine about 1 year ago Actions #15
@Jason Ish did you add a CI check with disable-ja3 ?
PA Updated by Philippe Antoine about 1 year ago Actions #16
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
PA Updated by Philippe Antoine about 1 year ago Actions #17
- Status changed from In Review to Resolved
https://github.com/OISF/suricata/pull/12927
I think we are now complete.
Remaining question is how we remain complete...
VJ Updated by Victor Julien 10 months ago Actions #18
- Status changed from Resolved to Closed