[Mini-NDN] IP-Routing Helper Fix

[Lan Wang (lanwang)]

Philipp,

Thank you very much for your contribution to Mini-NDN.  Saurab and Alex (and maybe Ashlesh) can review it.  I’m not sure what happened in the last review.  We follow the same review process for NFD and ndn-cxx.  If you have suggestions to improve it, we’re happy to listen to you.

Lan

> On Jul 21, 2020, at 5:43 AM, Philipp Moll <philipp.moll at itec.aau.at> wrote:
> 
> Hi everyone,
> 
> I just pushed a fix for the IP-Routing helper to Gerrit. The old version of the IP-Routing helper does not work for large topologies, such as the GEANT topology. This can be tested with `sudo python examples/ip_rounting_experiment.py topologies/geant.conf`. The new version fixes the problems and now, the IPRoutingHelper works in (hopefully) all topologies.
> 
> I made the path in March, but didn't find time to submit the path at that time. Now, I can't remember all the details, but pings were not successful when the path between two nodes was not symmetric (different paths for A-->B and B-->A). Technically, this should not matter, but for some reason, ping made troubles. I rewrote parts of the routing helper to prevent asymmetric paths (respectively asymmetric subpaths) and now pingall results in 100% success (`Results: 0% dropped (1980/1980 received)`) for the GEANT topology.
> 
> Long story short: the patch is working and I think everyone should be able to profit from that. However, I am currently busy with finishing my PhD, so, please spare me a review with hundreds of patch sets, as I experienced it with earlier patches.
> 
> Thanks,
> Philipp
> 
> _______________________________________________
> Mini-NDN mailing list
> Mini-NDN at lists.cs.ucla.edu
> http://www.lists.cs.ucla.edu/mailman/listinfo/mini-ndn
>