Skip to content

[dualtor] Leave icmp_responder running on dualtor topology#8909

Merged
zjswhhh merged 2 commits intosonic-net:masterfrom
zjswhhh:icmp_res_pub_mas
Jul 17, 2023
Merged

[dualtor] Leave icmp_responder running on dualtor topology#8909
zjswhhh merged 2 commits intosonic-net:masterfrom
zjswhhh:icmp_res_pub_mas

Conversation

@zjswhhh
Copy link
Contributor

@zjswhhh zjswhhh commented Jul 11, 2023

ADO#: 24560210

Description of PR

Summary:
Fixes # (issue)
Enforcing same behavior on dualtor, dualtor-aa and dualtor-mixed topology.

sign-off: Jing Zhang zhangjing@microsoft.com

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 201911
  • 202012
  • 202205

Approach

What is the motivation for this PR?

To enforce same behavior on dualtor, dualtor-aa, dualtor-mixed testbeds.

How did you do it?

How did you verify/test it?

Verified on dualtor testbed.

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

@zjswhhh zjswhhh merged commit cfcf8c9 into sonic-net:master Jul 17, 2023
@zjswhhh zjswhhh deleted the icmp_res_pub_mas branch July 17, 2023 17:52
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Jul 20, 2023
…-net#8909)

Summary:
Fixes # (issue)
Enforcing same behavior on dualtor, dualtor-aa and dualtor-mixed topology.

sign-off: Jing Zhang zhangjing@microsoft.com
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202205: #9064

mssonicbld pushed a commit that referenced this pull request Jul 21, 2023
Summary:
Fixes # (issue)
Enforcing same behavior on dualtor, dualtor-aa and dualtor-mixed topology.

sign-off: Jing Zhang zhangjing@microsoft.com
lolyu added a commit that referenced this pull request Jul 21, 2023
AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Jan 25, 2024
…-net#8909)

Summary:
Fixes # (issue)
Enforcing same behavior on dualtor, dualtor-aa and dualtor-mixed topology.

sign-off: Jing Zhang zhangjing@microsoft.com
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