Add Tunnel support for EOS - #3659
Conversation
|
See #3571, Ivan doesnt really wanted it in Netlab. |
|
@DanPartelly mm what i get for not looking at all the discussions before tinkering.. ;) |
Where did you get that impression from? I just told you to run away ;) @snuffy22 -- to solve this properly, you'd have to (A) figure out whether the "tunnel routes" applies only to OSPF or to all IGPs and then (B) add a device quirk that would set the corresponding flag in routing protocol data, followed by (C) modification of all routing protocol configuration templates. What you did was unfortunately "developing against the test", and even that would fail if we tested the tunnels in VRFs (not using VRFs for transport) |
|
@ipspace, guilty about for developing against the test. |
Probably context was off ? I got the impression literally from, and I quote: "You could start wireshare and figure out what's going on, but I would instead recommend to run away as fast as you can 🤦♂️ Also, this is not something I want to see implemented in netlab. " |
|
Did a little extra digging..
|
|
ISIS for my image returned 4 duplicates for any ICMP echo. Please test it manually. |
|
I will look into it in the next day or so and get back if i can produce same issue.. |
Thank you, and thank you for your prolific contributions to Netlab. |
|
Yep.. can confirm DUP. I tried expanding my lab to have a host on either side, h1 -> r2(GRE) -> dut -> (GRE)r3 -> h2 When pinging from h1 -> h2 I got 3 DUP responses. Now if I did a little python http server on h1 and had h2 download a test index.html.. Seems like it does not like ICMP related packets, feels like they need 'friends'. |
|
OK, @snuffy22 @DanPartelly, what should we do? Finalize this, declare the caveats, and merge, or forget it? |
|
Merge it, with documented caveats. |
There was a problem hiding this comment.
Pull request overview
This PR adds Arista EOS support to the existing tunnel.gre extra plugin, enabling GRE-over-IPv4 tunnels (including transport VRF) and documenting EOS-specific tunnel/OSPF caveats.
Changes:
- Added an EOS GRE tunnel configuration template for the
tunnel.greplugin. - Declared EOS GRE tunnel feature support (IPv4 + VRF) in the EOS device feature map.
- Enabled the EOS OSPFv2
tunnel routessetting and documented EOS GRE tunnel caveats.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| netsim/extra/tunnel/gre/eos.j2 | Adds EOS-specific GRE tunnel interface configuration (mode, underlay VRF, src/dst, PMTUD, TTL). |
| netsim/devices/eos.yml | Advertises EOS tunnel.gre feature support for ipv4 and vrf. |
| netsim/ansible/templates/ospf/eos.ospfv2.j2 | Enables tunnel routes in EOS OSPFv2 configuration to ensure tunnel-reachable routes are considered. |
| docs/plugins/tunnel.gre.md | Updates the GRE plugin supported-platform matrix to include Arista EOS capabilities. |
| docs/caveats.md | Documents EOS-specific GRE tunnel caveats (OSPFv2 tunnel routes, IS-IS behavior, and cEOS forwarding quirks). |
Add GRE support for EOS.
Passes tests as expected.
Do we want to update the
02-gre-vrf.ymltest ?