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

[smart_switch][dhcp_server] Fix query dhcp lease get unknown in smart switch by Cli #20642

Merged
merged 7 commits into from
Nov 1, 2024

Conversation

yaqiangz
Copy link
Contributor

@yaqiangz yaqiangz commented Oct 29, 2024

Why I did it

In smart switch, there is an issue that Cli query dhcp lease got unknow interface due to dpu fdb hasn't present in STATE_DB FDB_TABLE. Issue: #20155

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

How I did it

Query bridge fdb if there is no fdb record in STATE_DB

How to verify it

UT passed

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

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

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)

@yaqiangz
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yaqiangz yaqiangz marked this pull request as ready for review October 30, 2024 05:58
@yaqiangz
Copy link
Contributor Author

Hi @yxieca could you please help to merge this PR?

@yaqiangz yaqiangz changed the title [smart_switch][dhcp_server] Fix query dhcp lease get unknown in smart switch [smart_switch][dhcp_server] Fix query dhcp lease get unknown in smart switch by Cli Oct 31, 2024
@yxieca yxieca merged commit faaa69b into sonic-net:master Nov 1, 2024
21 checks passed
@yaqiangz
Copy link
Contributor Author

yaqiangz commented Nov 1, 2024

Hi @bingwang-ms could you please help to approve the backport request?

yaqiangz added a commit to yaqiangz/sonic-buildimage that referenced this pull request Nov 4, 2024
… switch by Cli (sonic-net#20642)

Why I did it
In smart switch, there is an issue that Cli query dhcp lease got unknow interface due to dpu fdb hasn't present in STATE_DB FDB_TABLE. Issue: sonic-net#20155

How I did it
Query bridge fdb if there is no fdb record in STATE_DB

How to verify it
UT passed
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.

5 participants