Skip to content

Fixing dynamic range modification UT#22561

Merged
StormLiangMS merged 2 commits intosonic-net:masterfrom
sudarshankumar4893:fixutmodifyiprange
May 9, 2025
Merged

Fixing dynamic range modification UT#22561
StormLiangMS merged 2 commits intosonic-net:masterfrom
sudarshankumar4893:fixutmodifyiprange

Conversation

@sudarshankumar4893
Copy link
Contributor

Why I did it

This PR adds the changes back from this PR: #22261
Along with 1 more change to fix the UT such that it only asserts if the template for dynamic neighbors exist.

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

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).

@sudarshankumar4893 sudarshankumar4893 marked this pull request as ready for review May 7, 2025 21:24
Signed-off-by: Sudarshan Kumar (from Dev Box) <[email protected]>
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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

@StormLiangMS StormLiangMS merged commit e62d234 into sonic-net:master May 9, 2025
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants