Skip to content

[action] [PR:18370] Fix BFD multihop IPv6 test case issue#18403

Merged
kevinskwang merged 1 commit intosonic-net:202411from
mssonicbld:cherry/202411/18370
May 20, 2025
Merged

[action] [PR:18370] Fix BFD multihop IPv6 test case issue#18403
kevinskwang merged 1 commit intosonic-net:202411from
mssonicbld:cherry/202411/18370

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

Description of PR

Summary:
The BFD multihop IPv6 session will flap during test.

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
  • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405
  • 202411

Approach

What is the motivation for this PR?

There is incorrect ovs flow rule that leaks the BFD multihop IPv6 packets to VM.

How did you do it?

Change ovs flow rule to only allow packets to go to PTF.

How did you verify/test it?

On physical testbed.

Any platform specific information?

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

Documentation

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: #18370

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@wsycqyz
Copy link
Copy Markdown
Contributor

wsycqyz commented May 16, 2025

Close pr and re-open to restart pr test.

@wsycqyz wsycqyz closed this May 16, 2025
@wsycqyz wsycqyz reopened this May 16, 2025
@mssonicbld
Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@wsycqyz
Copy link
Copy Markdown
Contributor

wsycqyz commented May 19, 2025

PR test still failed. Re-try

@kevinskwang kevinskwang merged commit fa05606 into sonic-net:202411 May 20, 2025
15 of 17 checks passed
auspham pushed a commit to auspham/sonic-mgmt that referenced this pull request May 30, 2025
)

```
8b16a64 (HEAD -> sync/202503, origin/sync/202503) Merge remote-tracking branch 'az_upstream/202412' into sync/202503
fe0ac4a (az_upstream/202412) Merge branch '202411' of https://github.com/sonic-net/sonic-mgmt into 202412
5c0cad1 Use is_alive over isAlive for py3 threads (sonic-net#18213) (sonic-net#18469)
b7f49b2 [action] [PR:16972] Update the sflow test to pass on the t0 topo which does not have portchannel (sonic-net#307)
64ecd5c Merge branch '202411' of https://github.com/sonic-net/sonic-mgmt into 202412
da50df8 Fix the generate_golden_config_db for multi-asic devices (sonic-net#18138)
a52cceb [action] [PR:18444] support t1-isolated-d56u2 in test_acl.py, test_stress_acl.py and test_qos_sai (sonic-net#305)
fa05606 fix bfd multihop test issue (sonic-net#18370) (sonic-net#18403)
c1f8f0f [action] [PR:18289] Fix ACL tests for t1-isolated-d28/128 (sonic-net#304)
```
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.

3 participants