Skip to content

[action] [PR:27906] copp: fix UDLD skip check to cover all Nokia H5/H6 fanout variants #1413

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.

Sign up for GitHub

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

Merged
mssonicbld merged 1 commit into from
Sep 14, 2026
Merged

[action] [PR:27906] copp: fix UDLD skip check to cover all Nokia H5/H6 fanout variants #1413

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 12 additions & 3 deletions tests/copp/test_copp.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@
import pytest
import json
import random
import re
import time
from collections import namedtuple

Expand Down Expand Up @@ -114,11 +115,19 @@ def test_policer(self, protocol, duthosts, enum_rand_one_per_hwsku_frontend_host
"""
# If fanout is running 7060x6 and running SONiC, the only supported action for UDLD is trap, which means
# UDLD packet will not be forwarded to DUT
# Nokia H5/H6 fanouts (all port-count variants, e.g. h5_32d, h5_64d, h5_64o, h6_64, h6_128) and
# Nexthop 4210 fanouts share the same Broadcom SAI/ASIC limitation and also only support trap
# (not forward) for UDLD.
if 'UDLD' == protocol:
for fanouthost in list(fanouthosts.values()):
if (fanouthost.get_fanout_os() == 'sonic' and fanouthost.facts["platform"]
in ['arista_7060x6_64pe', 'x86_64-nokia_ixr7220_h5_64o-r0', 'x86_64-nokia_ixr7220_h6_64-r0']):
pytest.skip("Skip UDLD test for Arista-7060x6 and Nokia-H5/H6 fanout without UDLD forward support")
fanout_platform = fanouthost.facts["platform"]
if fanouthost.get_fanout_os() == 'sonic' and (
fanout_platform == 'arista_7060x6_64pe'
or re.match(r'x86_64-nokia_ixr7220_h[56]_', fanout_platform)
or re.match(r'x86_64-nexthop_4210-', fanout_platform)
):
pytest.skip("Skip UDLD test for Arista-7060x6, Nokia-H5/H6 and Nexthop-4210 "
"fanout without UDLD forward support")

duthost = duthosts[enum_rand_one_per_hwsku_frontend_hostname]
namespace = DEFAULT_NAMESPACE
Expand Down