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

[202012] dhcprelay add Loopback0 as source interface in dualtor scenario #15744

Merged
merged 4 commits into from
Jul 25, 2023

Conversation

jcaiMR
Copy link
Contributor

@jcaiMR jcaiMR commented Jul 7, 2023

Why I did it

  1. sonic-build image side change to fix source interface selection in dual tor scenario.
    dhcprelay related PR:
    [202012] fix dual-tor relay-reply handing issue sonic-dhcp-relay#40

  2. Announce dhcprelay submodule to 3578eb3 (to invoke #40 PR)

Work item tracking
  • Microsoft ADO 24243215

How I did it

How to verify it

run test case, dhcp_relay/test_dhcpv6_relay.py

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)

@jcaiMR jcaiMR requested a review from yxieca July 12, 2023 08:02
@jcaiMR
Copy link
Contributor Author

jcaiMR commented Jul 12, 2023

@yxieca Hi Ying, would you please help approval and merge this PR.

@jcaiMR jcaiMR requested a review from qiluo-msft July 13, 2023 01:16
@qiluo-msft
Copy link
Collaborator

@jcaiMR Why there is no fix on master branch? If there should one, we need to fix master first, and backport to 202012 and later branches.

@jcaiMR
Copy link
Contributor Author

jcaiMR commented Jul 18, 2023

@jcaiMR Why there is no fix on master branch? If there should one, we need to fix master first, and backport to 202012 and later branches.

In 202012, dhcp_relay use different branch, the code base is huge different compare with master, so we have to use another PR for solution not cherry pick.
And this is the PR for master code fix: #15864

@qiluo-msft qiluo-msft merged commit 3c8ae45 into sonic-net:202012 Jul 25, 2023
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