goflow2 icon indicating copy to clipboard operation
goflow2 copied to clipboard

FIX: couble count of flow_process_nf_total

Open 0asys opened this issue 2 years ago • 1 comments

I was confused about the latest total processed flow counts - if I interpreting it right, the metric of flow_process_nf_total is counted double for v9 and v10:

https://github.com/netsampler/goflow2/blob/ee095a980f28a9cd9b478e12b9ff7d41ba533762/metrics/producer.go#L106

and in the recordCommonNetFlowMetrics function: https://github.com/netsampler/goflow2/blob/ee095a980f28a9cd9b478e12b9ff7d41ba533762/metrics/decoder.go#L89

0asys avatar Feb 07 '24 21:02 0asys

Good catch, it's also incremented in recordCommonNetFlowMetrics().

Thank you

lspgn avatar Feb 11 '24 17:02 lspgn

Hello, I think I prefer removing it from recordCommonNetFlowMetrics in #302, since NetFlow v5 is also emitting the metric in the same function. Thank you Will close this one.

lspgn avatar Mar 24 '24 03:03 lspgn