Skip to content

configure/qa-simulation: explicitly declare default - v1#15130

Closed
jufajardini wants to merge 1 commit intoOISF:main-8.0.xfrom
jufajardini:qa-simulation-build-info/v1
Closed

configure/qa-simulation: explicitly declare default - v1#15130
jufajardini wants to merge 1 commit intoOISF:main-8.0.xfrom
jufajardini:qa-simulation-build-info/v1

Conversation

@jufajardini
Copy link
Copy Markdown
Contributor

This is needed to generate the --build-info report.

Related to
Ticket #7885

(cherry picked from commit 41834f0)

Link to ticket: https://redmine.openinfosecfoundation.org/issues/
related to https://redmine.openinfosecfoundation.org/issues/7885

Describe changes:

  • explicitly add the no for when qa-simulation is disabled, otherwise this won't show in the build-info report

This is needed to generate the `--build-info` report.

Related to
Ticket OISF#7885

(cherry picked from commit 41834f0)
@jufajardini jufajardini requested a review from a team as a code owner March 31, 2026 12:44
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 31, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.59%. Comparing base (a4aa865) to head (dc9c3c1).
⚠️ Report is 7 commits behind head on main-8.0.x.

Additional details and impacted files
@@             Coverage Diff             @@
##           main-8.0.x   #15130   +/-   ##
===========================================
  Coverage       83.59%   83.59%           
===========================================
  Files            1011     1011           
  Lines          266822   266822           
===========================================
+ Hits           223048   223057    +9     
+ Misses          43774    43765    -9     
Flag Coverage Δ
fuzzcorpus 64.02% <ø> (+0.02%) ⬆️
livemode 18.74% <ø> (-0.08%) ⬇️
pcap 44.58% <ø> (-0.01%) ⬇️
suricata-verify 64.82% <ø> (-0.02%) ⬇️
unittests 58.83% <ø> (-0.01%) ⬇️

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@suricata-qa
Copy link
Copy Markdown

Information: QA skipped due to no C or rust code changed detected. Set to force a run.

Pipeline = code

@victorjulien victorjulien added this to the 8.0 milestone Mar 31, 2026
@victorjulien
Copy link
Copy Markdown
Member

Merged in #15139, thanks!

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