Project

General

Profile

Actions

Task #6962

closed

yaml: unify 0 stats counter config option terminology

Added by Juliana Fajardini Reichow 7 months ago. Updated 6 months ago.

Status:
Closed
Priority:
Normal
Target version:
Effort:
Difficulty:
Label:

Description

When doing the research for #5976 I failed to see that we also have a config stats option for the human-readable logs to output 0 counters.
Due to not seeing this before, we now have two different setting names for basically the same thing, but in different logs:
zero-valued-counters for EVE https://github.com/OISF/suricata/blob/master/suricata.yaml.in#L319 and
null-values for stats.log https://github.com/OISF/suricata/blob/master/suricata.yaml.in#L444C9-L444C20.

I think we should use the same terminology, and that I should change the recently added one to null-values, as this one has been around for longer.

Opened a ticket so this can be registered as a lesson learned (hopefully!).


Related issues 1 (0 open1 closed)

Related to Suricata - Feature #5976: eve/stats: allow hiding counters whose value is 0ClosedJuliana Fajardini ReichowActions
Actions #1

Updated by Juliana Fajardini Reichow 7 months ago

  • Related to Feature #5976: eve/stats: allow hiding counters whose value is 0 added
Actions #2

Updated by Juliana Fajardini Reichow 7 months ago

  • Status changed from New to In Review
Actions #3

Updated by Juliana Fajardini Reichow 6 months ago

  • Status changed from In Review to Closed
Actions

Also available in: Atom PDF