Skip to content
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

Verbosity maybe its too much or too little #393

Closed
gabrielmocan opened this issue Dec 20, 2022 · 1 comment
Closed

Verbosity maybe its too much or too little #393

gabrielmocan opened this issue Dec 20, 2022 · 1 comment
Assignees
Labels
Feature request Feature request

Comments

@gabrielmocan
Copy link

Hi Pete, I have a question about verbosity.

Back in the days of 1.6.x we had a consistency across nfcapd and sfcapd binaries.

A very useful information, which is displayed every minute, shows summary per exporter. Like below:

2022-12-20T11:42:00.004011279Z Total ignored packets: 0
2022-12-20T11:43:00.049420678Z Ident: 'wwwwwww' Flows: 8359, Packets: 9738, Bytes: 7285286, Sequence Errors: 3, Bad Packets: 0
2022-12-20T11:43:00.050152255Z Ident: 'zzzzzzzz' Flows: 9762, Packets: 15236, Bytes: 16410837, Sequence Errors: 3, Bad Packets: 0
2022-12-20T11:43:00.050296065Z Ident: 'yyyyyyy' Flows: 0, Packets: 0, Bytes: 0, Sequence Errors: 2, Bad Packets: 0
2022-12-20T11:43:00.051099732Z Ident: 'xxxxxxx' Flows: 44813, Packets: 128244, Bytes: 117486341, Sequence Errors: 0, Bad Packets: 0

Now that behaviour changed on 1.7, meaning that I have to use "-v" flag to display this info. Ok so far, but 2 things comes in:

  1. sometimes I have a bulk of random information between the regular minute-log:

2022-12-20T11:41:00.143998745Z File Block Header:
2022-12-20T11:41:00.144014081Z Type = 3
2022-12-20T11:41:00.144029060Z Size = 32
2022-12-20T11:41:00.144032957Z NumRecords = 1
2022-12-20T11:41:00.144036213Z
2022-12-20T11:41:00.144039563Z File Block Header:
2022-12-20T11:41:00.144043266Z Type = 3
2022-12-20T11:41:00.144046485Z Size = 45132
2022-12-20T11:41:00.144049570Z NumRecords = 249
2022-12-20T11:41:00.144052803Z
2022-12-20T11:41:00.144056183Z File Block Header:
2022-12-20T11:41:00.144064897Z Type = 3
2022-12-20T11:41:00.144068675Z Size = 435240
2022-12-20T11:41:00.144072068Z NumRecords = 2430
2022-12-20T11:41:00.144075296Z
2022-12-20T11:41:00.144078570Z File Block Header:
2022-12-20T11:41:00.144081721Z Type = 3
2022-12-20T11:41:00.144084969Z Size = 691556
2022-12-20T11:41:00.144088305Z NumRecords = 3743
2022-12-20T11:41:00.144091464Z
.
.
.

  1. Using -v option on sfcapd makes it extremely verbose, basically logging EVERY record that comes in.

So I ask, could we go back to the old behaviour of 1.6, just to show the per minute summary for both nfcapd and sfcapd without having to pass "-v" flag?

Best,
Gabe.

@phaag
Copy link
Owner

phaag commented Dec 22, 2022

Ok - fair point! I will rework the logging.

@phaag phaag closed this as completed in 164aa3c Dec 22, 2022
@phaag phaag added the Feature request Feature request label Dec 22, 2022
@phaag phaag self-assigned this Dec 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature request Feature request
Projects
None yet
Development

No branches or pull requests

2 participants