Rare project developers

[Alexander Gall <>]

On Thu, 24 Feb 2022 09:54:00 +0100, Frédéric LOUI <> 
said:

> Hi Alex,
> Feel free to apply your fix to bf_forwarder.py and the p4 code.
 
> The place where subintf counters are processed in bf_forwarde.py should be 
> _only_  here:
> https://bitbucket.software.geant.org/projects/RARE/repos/rare/browse/bfrt_python/rare/bf_subifcounter/__init__.py

Maybe it should, but it obviously isn't :( To avoid duplication of
code, I think we'll have to modify the counter modules with an API to
share this code with the SNMP module in a thread-safe manner. Other
suggestions?

-- 
Alex


> Thanks for this feedback !
> Frederic

>> Le 24 févr. 2022 à 09:26, Alexander Gall <> a écrit :
>> 
>> A possible fix would be to index pkt_out_stats with the VLAN ID
>> instead of the source_id (which is always 0). It would be even nicer
>> if we could fix the counter already in the P4 code rather than leaving
>> this to the forwarder. One problem with this is that we have to
>> replicate the counter-fix in multiple places, i.e. the SNMP code and
>> wherever the forwarder pushes the same counter to freerou