Skip to content

Extend BGP Confederation schema with member-ASN ranges and loose-path-validation.#1482

Open
rszarecki wants to merge 5 commits into
openconfig:masterfrom
rszarecki:BGP_Confed
Open

Extend BGP Confederation schema with member-ASN ranges and loose-path-validation.#1482
rszarecki wants to merge 5 commits into
openconfig:masterfrom
rszarecki:BGP_Confed

Conversation

@rszarecki
Copy link
Copy Markdown
Contributor

Change Scope

Extend BGP Confederation member-ASN ranges and loose-path-validation.

In large BGP Confederated network it is impractical to explicitly list in configuration all potentially 100's of member-AS. It is also not required for implementaion to be aware of all member-ASN. Implementation only need to know member-ASN it has configured confederation external BGP session.

  • The member-as-range structure allows for efficient listing large number of member-ASN if they are allocated in structured way e.g. gotm 4B ASN private space.
  • The 'loose-as-validation' boolean allows to explicitly control implementation behaviour regarding AS_PATH attribute validdeation. This is needed since different existing implementaions has disticnt implicit and non-configurable behaviours.

Platform Implementations

AS_PATH validation:

implementation implicit behaviour
Arista EoS loose. Path is valid if non-listed ASN is in BGP_CONFEDERATION_SEQUENCE object.
Cisco IOS loose. Path is valid if non-listed ASN is in BGP_CONFEDERATION_SEQUENCE object.
Juniper strict. Path is invalid if non-listed ASN is in BGP_CONFEDERATION_SEQUENCE object. Error handling may reset BGP session.
Nokia SR-OS strict. Path is invalid if non-listed ASN is in BGP_CONFEDERATION_SEQUENCE object. Error handling may reset BGP session. (to be confirmed)
Quagga/FRR loose. Path is valid if non-listed ASN is in BGP_CONFEDERATION_SEQUENCE object

member-as-range

This is a new feature.

Tree View

module: openconfig-network-instance
  +--rw network-instances
     +--rw network-instance* [name]
        +--rw protocols
           +--rw protocol* [identifier name]
              +--rw bgp
                 +--rw global
                    +--rw confederation
                       +--rw config
                       |  +--rw identifier?              oc-inet:as-number
                       |  +--rw member-as*               oc-inet:as-number
+                      |  +--rw loose-path-validation?   boolean
                       +--ro state
                       |  +--ro identifier?              oc-inet:as-number
                       |  +--ro member-as*               oc-inet:as-number
+                      |  +--ro loose-path-validation?   boolean
+                       +--rw member-as-ranges
+                          +--rw member-as-range* [first-asn last-asn]
+                             +--rw first-asn    -> ../config/first-asn
+                             +--rw last-asn     -> ../config/last-asn
+                             +--rw config
+                             |  +--rw first-asn?   oc-inet:as-number
+                             |  +--rw last-asn?    oc-inet:as-number
+                             +--ro state
+                                +--ro first-asn?   oc-inet:as-number
+                                +--ro last-asn?    oc-inet:as-number

@rszarecki rszarecki requested a review from a team as a code owner April 28, 2026 00:45
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the OpenConfig BGP models to version 9.10.0, introducing schema support for BGP Confederation member-ASN ranges and loose-path-validation. The changes include a new grouping for ASN ranges and a configuration leaf for path validation behavior. Feedback suggests adding a 'must' constraint to ensure ASN ranges are valid and provides corrections for typos in the documentation strings.

Comment thread release/models/bgp/openconfig-bgp-global.yang
Comment thread release/models/bgp/openconfig-bgp-global.yang
@OpenConfigBot
Copy link
Copy Markdown

OpenConfigBot commented Apr 28, 2026

No major YANG version changes in commit e1acc8a

rszarecki and others added 2 commits April 27, 2026 17:51
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@rszarecki rszarecki changed the title cleanup ols changes Extend BGP Confederation schema with member-ASN ranges and loose-path-validation. Apr 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants