Skip to content

[action] [PR:18787] Xfail test_dir_bcast.py due to known issue on Broadcom platform#18825

Merged
mssonicbld merged 1 commit intosonic-net:202505from
mssonicbld:cherry/202505/18787
Jun 5, 2025
Merged

[action] [PR:18787] Xfail test_dir_bcast.py due to known issue on Broadcom platform#18825
mssonicbld merged 1 commit intosonic-net:202505from
mssonicbld:cherry/202505/18787

Conversation

@mssonicbld
Copy link
Collaborator

Description of PR

Summary:
Fixes # (issue)
t0-118 DUTs running ipfwd/test_dir_bcast.py do not seem to be broadcasting (ip) packets to all VLAN members when the packet is sent to the VLAN broadcast IP.

Looking at the counters CLI output in sonic, "Broadcast Packets Transmitted" is not incrementing for a few interfaces in the VLAN

The test passes if the secondary_subnet ip address is removed from the first vlan in the t0-118 topo file.

But for now we enabled secondary subnet for most of t0 topologies #18399.

We have a CSP to track this issue with Broadcom, for now, xfail test_dir_bcast.py on Broadcom T0 testbeds.
File a github issue to track it #18786, once the issue is fixed, we close this issue can disable xfail.

Type of change

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

Back port request

  • 202205
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505

Approach

What is the motivation for this PR?

Xfail test_dir_bcast.py on Broadcom T0 testbeds.

How did you do it?

Add xfail section in conditional mark file

How did you verify/test it?

Run test_dir_bcast on Broadcom T0 testbed to see if it's xfailed.

Any platform specific information?

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

Documentation

…c-net#18787)

What is the motivation for this PR?
Xfail test_dir_bcast.py on Broadcom T0 testbeds.

How did you do it?
Add xfail section in conditional mark file

How did you verify/test it?
Run test_dir_bcast on Broadcom T0 testbed to see if it's xfailed.
@mssonicbld
Copy link
Collaborator Author

Original PR: #18787

@mssonicbld
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit 24cb9c0 into sonic-net:202505 Jun 5, 2025
17 checks 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.

2 participants