Skip to content

Added new test case to check docker-dhcp-relay file generation when secondary subnets are present#17263

Merged
yxieca merged 16 commits intosonic-net:masterfrom
shbalaku-microsoft:add-test-cases-for-dhcp-relay-file-changes-when-secondary-subnets-are-present
Mar 5, 2024
Merged

Added new test case to check docker-dhcp-relay file generation when secondary subnets are present#17263
yxieca merged 16 commits intosonic-net:masterfrom
shbalaku-microsoft:add-test-cases-for-dhcp-relay-file-changes-when-secondary-subnets-are-present

Conversation

@shbalaku-microsoft
Copy link
Contributor

@shbalaku-microsoft shbalaku-microsoft commented Nov 22, 2023

Why I did it

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

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)

@zhangyanzhao
Copy link

zhangyanzhao commented Feb 5, 2024

@shbalaku-microsoft is this PR to add test cases or feature code? If this is test case, then it should go to sonic-mgmt repo. add @prsunny to double check.

prsunny
prsunny previously approved these changes Feb 6, 2024
Copy link
Contributor

@prsunny prsunny 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 prsunny requested a review from kellyyeh February 6, 2024 01:37
@prsunny
Copy link
Contributor

prsunny commented Mar 4, 2024

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@prsunny
Copy link
Contributor

prsunny commented Mar 5, 2024

@shbalaku-microsoft , please check the build failure

@yxieca yxieca merged commit a6a8d19 into sonic-net:master Mar 5, 2024
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.

5 participants