Open stal76 opened 1 month ago
Also I would suggest splitting commit into two: one for the new functionality and the other for the test.
Overall the PR is ok but I would prefer to hold it until shared memory for counters and counter identifier allocator PR would be merged. Also I am not sure if available amount of counters is big enough so will test it on our production before.
Merging functionality and tests is Ok but please update the PR according to comments.
Now one can view the number and size of packets for each entry in the route.
Added a new autotest - 075_counters_rib, in the autotest.yaml one can see an example of statistics output.