SRv6 on Junos

[ipspace]

• SRv6 locator configuration
• SRv6 locator advertisement with IS-IS
• Layer-3 services (IPv4 + IPv6) with SRv6

[jbemmel]

Perhaps if 'bgp' in srv6.igp or 'ebgp' in srv6.igp ?

FRR uses {% if bgp.srv6 is defined %}

[jbemmel]

Looking at the code, I saw at least 3 different ways to check for when to enable BGP with SRv6 - this may be a good moment to consolidate

e.g. FRR has {% if 'bgp' in srv6.igp|default([]) or srv6.bgp|default(False) %} at the top and then something else below, etc.

[ipspace]

Looking at the code, I saw at least 3 different ways to check for when to enable BGP with SRv6 - this may be a good moment to consolidate

Agreed. Will do that after merging this one. You know how lovely it is to trace a bug in the code to a totally unrelated PR ;)

[Copilot]

Copilot review overview

🔵 Needs a closer look

The Junos SRv6 template has two unresolved moderate issues.

Review effort: Lite
Findings: None

Resolved since last review (3)

• Add the required SRv6 locator prefix statement
• Declare Junos SRv6 VPN capability
• Declare Junos SRv6 VPN capability

Previously missed (1)

In code that hasn't changed since last review

Missing SRv6 source address from IPv6 loopback

netsim/​ansible/​templates/​srv6/​junos.j2:3

This srv6 block never configures Junos's source-address, so SRv6 encapsulation/service SIDs have no source IPv6 address; the SRv6 template contract requires deriving it from loopback.ipv6 (as the other SRv6 templates do). Add the address-only loopback value here.

This issue also appears on line 11 of the same file.