-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Live tests/v5 #11074
Live tests/v5 #11074
Conversation
In the default config iface bypass support is not enabled, and storage API not initialized for it. Using it will lead to a crash. This commit first checks if the device storage API is initialized. Bug: OISF#7022.
Commands would leave use_cnt incremented, never decrementing them. This would lead to a asserting triggering at shutdown. Bug: OISF#7020.
Runs with ASAN.
To increase code coverage.
Asan and coverage jobs.
Should the old PRs get closed ? |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #11074 +/- ##
==========================================
+ Coverage 83.63% 84.06% +0.43%
==========================================
Files 922 922
Lines 250321 250324 +3
==========================================
+ Hits 209349 210428 +1079
+ Misses 40972 39896 -1076
Flags with carried forward coverage won't be shown. Click here to find out more. |
@@ -0,0 +1,155 @@ | |||
#!/bin/bash | |||
|
|||
# Script to test live IDS capabilities for OCAO. Starts a ping, starts suricata, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whats OCAO?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo, meant to say "PCAP"
Information: QA ran without warnings. Pipeline 20620 |
replaced by #11075 |
#11073, plus a libpcap live job based on the af-packet script.