Conversation
|
🏗 Preview site has been deployed to: |
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR updates the L2 EVPN tutorial to support SR Linux version 25.10. The changes modernize the lab configuration and documentation with updated versions and corrected syntax.
Key changes:
- Updated container images to newer versions (srlinux 25.10, containerlab 0.71.1, docker-ce 27.5.1)
- Corrected device type names to use hyphenated format (ixrd2 → ixr-d2, ixrd3 → ixr-d3)
- Fixed BGP policy syntax to use bracket notation (export-policy all → export-policy [ all ])
- Enhanced documentation formatting with embed-result divs and navigation links
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| labs/evpn01.clab.yml | Updated srlinux image to 25.10, network-multitool image path, and corrected device type names to use hyphenated format |
| docs/tutorials/l2evpn/intro.md | Updated version information, improved code block formatting with embed-result styling, added navigation link, and updated command presentation |
| docs/tutorials/l2evpn/fabric.md | Corrected BGP export-policy and import-policy syntax throughout all configuration examples to use bracket notation, added navigation link |
| docs/tutorials/l2evpn/evpn.md | Corrected BGP export-policy and import-policy syntax throughout overlay BGP configuration examples |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| | **Packet captures** | [EVPN IMET routes exchange][capture-imets], [RT2 routes exchange with ICMP in datapath][capture-rt2-datapath] | | ||
| | **Main ref documents** | [RFC 7432 - BGP MPLS-Based Ethernet VPN](https://datatracker.ietf.org/doc/html/rfc7432)<br/>[RFC 8365 - A Network Virtualization Overlay Solution Using Ethernet VPN (EVPN)](https://datatracker.ietf.org/doc/html/rfc8365)<br/>[Nokia 7220 SR Linux Advanced Solutions Guide][adv-sol-guide-evpn-l2]<br/>[Nokia 7220 SR Linux EVPN-VXLAN Guide][evpn-vxlan-guide] | | ||
| | **Version information**[^1] | [`containerlab:0.48.6`][clab-install], [`srlinux:23.10.1`][srlinux-container], [`docker-ce:23.0.3`][docker-install] | | ||
| | **Version information**[^1] | [`containerlab:0.71.1`][clab-install], [`srlinux:25.10`][srlinux-container], [`docker-ce:27.5.1`][docker-install] | |
There was a problem hiding this comment.
The documented srlinux version '25.10' differs from the PR title which mentions '25.10.1'. This inconsistency should be resolved to match either the actual image tag being used or the intended version. If '25.10.1' is the target version, update this documentation and the image tag in evpn01.clab.yml accordingly.
Suggested change
| | **Version information**[^1] | [`containerlab:0.71.1`][clab-install], [`srlinux:25.10`][srlinux-container], [`docker-ce:27.5.1`][docker-install] | | |
| | **Version information**[^1] | [`containerlab:0.71.1`][clab-install], [`srlinux:25.10.1`][srlinux-container], [`docker-ce:27.5.1`][docker-install] | |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.