Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[frr-mgmt-framework]: VXLAN EVPN should support advertise-svi-ip #21336

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bradh352
Copy link
Contributor

@bradh352 bradh352 commented Jan 6, 2025

Why I did it

advertise-svi-ip allows the switch to participate in the L2 VXLAN via its unique IP address.

https://docs.frrouting.org/en/latest/bgp.html#evpn-advertise-svi-ip

Work item tracking

How I did it

Updated yang models, frr-mgmt-framework, jinja2 templates, and test cases.

How to verify it

Test cases have been updated for verification purposes.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202411

Tested branch (Please provide the tested image version)

  • master as of 20250106
  • 202411

Description for the changelog

[frr-mgmt-framework]: VXLAN EVPN should support advertise-svi-ip

Link to config_db schema for YANG module changes

No relevant section exists in Configuration.md to document change.

A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: Brad House (@bradh352)

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@bradh352 bradh352 force-pushed the bradh352/advertise-svi-ip branch from 105cdad to a8c3c42 Compare January 8, 2025 12:10
@bradh352
Copy link
Contributor Author

bradh352 commented Jan 8, 2025

rebased against master to force rebuild

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

advertise-svi-ip allows the switch to participate in the L2 VXLAN
via its unique IP address.

Signed-off-by: Brad House (@bradh352)
@bradh352 bradh352 force-pushed the bradh352/advertise-svi-ip branch from a8c3c42 to 5cc5b1d Compare January 9, 2025 13:05
@bradh352
Copy link
Contributor Author

bradh352 commented Jan 9, 2025

rebased to force rebuild due to CI test failures (unrelated to this PR)

@bradh352
Copy link
Contributor Author

bradh352 commented Jan 9, 2025

@qiluo-msft @lguohan please review

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

bradh352 added a commit to bradh352/sonic-buildimage that referenced this pull request Jan 17, 2025
…onic-net#21336]

advertise-svi-ip allows the switch to participate in the L2 VXLAN
via its unique IP address.

Signed-off-by: Brad House (@bradh352)
bradh352 added a commit to bradh352/sonic-buildimage that referenced this pull request Jan 17, 2025
…onic-net#21336)

advertise-svi-ip allows the switch to participate in the L2 VXLAN
via its unique IP address.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Jan 17, 2025
…onic-net#21336]

advertise-svi-ip allows the switch to participate in the L2 VXLAN
via its unique IP address.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Jan 18, 2025
…onic-net#21336)

advertise-svi-ip allows the switch to participate in the L2 VXLAN
via its unique IP address.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Jan 18, 2025
…onic-net#21336]

advertise-svi-ip allows the switch to participate in the L2 VXLAN
via its unique IP address.

Signed-off-by: Brad House (@bradh352)
Copy link
Contributor

@StormLiangMS StormLiangMS left a comment

Choose a reason for hiding this comment

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

LGTM @prsunny could you help to take a look?

github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Jan 20, 2025
…onic-net#21336]

advertise-svi-ip allows the switch to participate in the L2 VXLAN
via its unique IP address.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Jan 21, 2025
…onic-net#21336]

advertise-svi-ip allows the switch to participate in the L2 VXLAN
via its unique IP address.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Jan 21, 2025
…onic-net#21336]

advertise-svi-ip allows the switch to participate in the L2 VXLAN
via its unique IP address.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Jan 21, 2025
…onic-net#21336)

advertise-svi-ip allows the switch to participate in the L2 VXLAN
via its unique IP address.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Jan 21, 2025
…onic-net#21336]

advertise-svi-ip allows the switch to participate in the L2 VXLAN
via its unique IP address.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Jan 23, 2025
…onic-net#21336]

advertise-svi-ip allows the switch to participate in the L2 VXLAN
via its unique IP address.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Jan 23, 2025
…onic-net#21336)

advertise-svi-ip allows the switch to participate in the L2 VXLAN
via its unique IP address.

Signed-off-by: Brad House (@bradh352)
@lguohan lguohan added the YANG YANG model related changes label Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
YANG YANG model related changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants