Skip to content

[YANG] sonic-neigh.yang need support portchannel #22074

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

Merged
merged 5 commits into from
Mar 20, 2025

Conversation

wen587
Copy link
Contributor

@wen587 wen587 commented Mar 19, 2025

Why I did it

NEIGH need to support PORTCHANNEL for t1 topology when VLAN is non-existent.

Work item tracking
  • Microsoft ADO (number only): 31872308

How I did it

Add portchannel to port as an alllowed leafref

How to verify it

unit test

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

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

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

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

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@wen587 wen587 marked this pull request as ready for review March 19, 2025 07:37
prsunny
prsunny previously approved these changes Mar 19, 2025
qiluo-msft
qiluo-msft previously approved these changes Mar 19, 2025
@wen587 wen587 dismissed stale reviews from qiluo-msft and prsunny via 3d59e9c March 19, 2025 22:57
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

kperumalbfn
kperumalbfn previously approved these changes Mar 19, 2025
Copy link
Contributor

@kperumalbfn kperumalbfn left a comment

Choose a reason for hiding this comment

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

LGTM

@kperumalbfn kperumalbfn enabled auto-merge (squash) March 19, 2025 23:19
auto-merge was automatically disabled March 20, 2025 00:07

Head branch was pushed to by a user without write access

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@wangxin
Copy link
Contributor

wangxin commented Mar 20, 2025

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kperumalbfn kperumalbfn enabled auto-merge (squash) March 20, 2025 05:11
@kperumalbfn kperumalbfn merged commit cafaf4d into sonic-net:master Mar 20, 2025
24 checks passed
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202411: #22095

yanjundeng pushed a commit to yanjundeng/sonic-buildimage that referenced this pull request Apr 23, 2025
[YANG] sonic-neigh.yang need support portchannel
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.

7 participants