Skip to content

[action] [PR:4179] Skip IP range duplicate check in validate_bgp_peer_group for different vnets#4215

Merged
mssonicbld merged 1 commit intosonic-net:202511from
mssonicbld:cherry/202511/4179
Jan 22, 2026
Merged

[action] [PR:4179] Skip IP range duplicate check in validate_bgp_peer_group for different vnets#4215
mssonicbld merged 1 commit intosonic-net:202511from
mssonicbld:cherry/202511/4179

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

What I did

Skip IP range duplicate check in validate_bgp_peer_group for different vnets
Related issue: #2145

How I did it

Check duplicates only when both vnet_name and ip_range match

How to verify it

testbed tests and unit tests

…t vnets

<!--
    Please make sure you've read and understood our contributing guidelines:
    https://github.com/Azure/SONiC/blob/gh-pages/CONTRIBUTING.md

    ** Make sure all your commits include a signature generated with `git commit -s` **

    If this is a bug fix, make sure your description includes "closes #xxxx",
    "fixes #xxxx" or "resolves #xxxx" so that GitHub automatically closes the related
    issue when the PR is merged.

    If you are adding/modifying/removing any command or utility script, please also
    make sure to add/modify/remove any unit tests from the tests
    directory as appropriate.

    If you are modifying or removing an existing 'show', 'config' or 'sonic-clear'
    subcommand, or you are adding a new subcommand, please make sure you also
    update the Command Line Reference Guide (doc/Command-Reference.md) to reflect
    your changes.

    Please provide the following information:
-->

#### What I did
Skip IP range duplicate check in `validate_bgp_peer_group` for different vnets
Related issue: sonic-net#2145

#### How I did it
Check duplicates only when both `vnet_name` and `ip_range` match

#### How to verify it
testbed tests and unit tests
@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: #4179

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit e4881b0 into sonic-net:202511 Jan 22, 2026
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant