Skip to content

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

Merged
kperumalbfn merged 5 commits intosonic-net:masterfrom
wen587:neigh_yang
Mar 20, 2025
Merged

[YANG] sonic-neigh.yang need support portchannel#22074
kperumalbfn merged 5 commits intosonic-net:masterfrom
wen587:neigh_yang

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

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
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

@azure-pipelines
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

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@wangxin
Copy link
Contributor

wangxin commented Mar 20, 2025

/azp run Azure.sonic-buildimage

@azure-pipelines
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

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