Skip to content

net/frr: Add per-neighbor local-as no-prepend/replace-as options - #5688

Open
clarkchentw wants to merge 1 commit into
opnsense:masterfrom
clarkchentw:frr-neighbor-local-as-no-prepend-replace-as
Open

net/frr: Add per-neighbor local-as no-prepend/replace-as options#5688
clarkchentw wants to merge 1 commit into
opnsense:masterfrom
clarkchentw:frr-neighbor-local-as-no-prepend-replace-as

Conversation

@clarkchentw

Copy link
Copy Markdown

Important notices

Before you submit a pull request, we ask you kindly to acknowledge the following:

If AI was used, please disclose:

  • Model used: grok-4.6 (development), gpt-5.6-sol (review)
  • Extent of AI involvement: grok-4.6 assisted with implementing the model/form/template changes, validation constraint, and changelog. gpt-5.6-sol performed an independent code review. Changes were reviewed.

Describe the problem

Neighbor-level local-as was added in #5308, but FRR's no-prepend and replace-as modifiers are still missing. Without them, the configured Local AS is prepended to received AS-Paths, and the real AS remains in outbound updates. That is often not the desired behavior for ASN migrations or provider peering such as AWS Direct Connect.


Describe the proposed solution

This pull request adds optional per-neighbor Local AS modifiers:

  • Add localasoptions to the BGP neighbor model (no-prepend, no-prepend replace-as)
  • Add an advanced Local AS Options dropdown next to Local AS
  • Require Local AS when Local AS Options are set
  • Render neighbor <address> local-as <asn> [no-prepend [replace-as]] in bgpd.conf when configured

The change is backward compatible: no extra keywords are emitted unless explicitly configured.


Related issue

Closes #5687

@Monviech Monviech self-assigned this Aug 30, 2026
@Monviech Monviech added the feature Adding new functionality label Aug 30, 2026
@Monviech

Copy link
Copy Markdown
Member

This will most likely make it in in some form or another at some point.

Currently Im working on cleaning up the BGP templates, so new features should be deferred until that work has been done.

The current features should work as expected before adding new ones.

So I'm putting this in the queue but it might take a while still.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Adding new functionality

Development

Successfully merging this pull request may close these issues.

os-frr: add neighbor local-as no-prepend/replace-as options

2 participants