[Nexthop] NH-5010 Platform Support by roy-nexthop · Pull Request #24169 · sonic-net/sonic-buildimage

roy-nexthop · 2025-09-30T19:32:22Z

Why I did it

Adds support for the Nexthop NH-5010 platform

Work item tracking

Microsoft ADO (number only):

How I did it

Add platform changes to:

device/nexthop
platform/broadcom/sonic-platform-modules-nexthop

How to verify it

Which release branch to backport (provide reason below if selected)

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

mssonicbld · 2025-09-30T19:32:30Z

/azp run Azure.sonic-buildimage

azure-pipelines · 2025-09-30T19:32:42Z

Azure Pipelines successfully started running 1 pipeline(s).

mssonicbld · 2025-09-30T21:21:27Z

/azp run Azure.sonic-buildimage

azure-pipelines · 2025-09-30T21:21:43Z

Azure Pipelines successfully started running 1 pipeline(s).

mssonicbld · 2025-10-07T20:11:29Z

/azp run Azure.sonic-buildimage

azure-pipelines · 2025-10-07T20:11:35Z

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

mssonicbld · 2025-10-07T20:22:57Z

/azp run Azure.sonic-buildimage

azure-pipelines · 2025-10-07T20:23:09Z

Azure Pipelines successfully started running 1 pipeline(s).

mssonicbld · 2025-10-07T20:33:38Z

/azp run Azure.sonic-buildimage

azure-pipelines · 2025-10-07T20:33:50Z

Azure Pipelines successfully started running 1 pipeline(s).

device/nexthop/x86_64-nexthop_5010-r0/NH-5010-F-O32-C32/port_config.ini

eswaran-nexthop · 2025-10-07T20:51:26Z

@arlakshm to review

device/nexthop/x86_64-nexthop_5010-r0/NH-5010-F-O32-C32/BALANCED/buffers_defaults_t2.j2

device/nexthop/x86_64-nexthop_5010-r0/NH-5010-F-O64/port_config.ini

arlakshm · 2025-10-07T21:03:20Z

...oadcom/sonic-platform-modules-nexthop/common/etc/systemd/system/swss.service.d/override.conf

@@ -0,0 +1,3 @@
+[Unit]
+Wants=pmon.service


why do we need this change?

One of the main reasons is to ensure thermal control starts before swss begins enabling interfaces.

this change means the swss will not start till pmon is started? @judyjoseph for viz...

@arlakshm correct

Another reason we added this is because if xcvrd programs the xcvrs after the port is enabled the link could flap. Ideally there is a generic solution where orchagent keeps the port disabled until xcvrd has programmed the xcvr. Until something like that is implemented we're adding this as a temporary improvement for Nexthop platforms.

device/nexthop/x86_64-nexthop_5010-r0/NH-5010-F-O64/nh5010-default.bcm

mssonicbld · 2025-10-15T17:01:04Z

/azp run Azure.sonic-buildimage

azure-pipelines · 2025-10-15T17:01:19Z

Azure Pipelines successfully started running 1 pipeline(s).

mssonicbld · 2025-10-21T19:21:35Z

/azp run Azure.sonic-buildimage

azure-pipelines · 2025-10-21T19:21:49Z

Azure Pipelines successfully started running 1 pipeline(s).

mssonicbld · 2025-10-21T19:22:40Z

/azp run Azure.sonic-buildimage

azure-pipelines · 2025-10-21T19:22:54Z

Azure Pipelines successfully started running 1 pipeline(s).

mssonicbld · 2025-10-22T21:21:11Z

/azp run Azure.sonic-buildimage

azure-pipelines · 2025-10-22T21:21:25Z

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines · 2025-10-24T16:40:07Z

Azure Pipelines successfully started running 1 pipeline(s).

arlakshm · 2025-10-28T23:33:47Z

...oadcom/sonic-platform-modules-nexthop/common/etc/systemd/system/swss.service.d/override.conf

@@ -0,0 +1,3 @@
+[Unit]
+Wants=pmon.service


this change means the swss will not start till pmon is started? @judyjoseph for viz...

arlakshm · 2025-10-28T23:37:01Z

device/nexthop/x86_64-nexthop_5010-r0/NH-5010-F-O64/BALANCED/buffers_defaults_t2.j2

+{%- macro generate_qos_bypass_port_list(PORT_QOS_BYPASS) %}
+    {# Generate list of ports #}
+    {%- for port_idx in range(256, 261, 4) %}
+        {%- if PORT_QOS_BYPASS.append("Ethernet%d" % (port_idx)) %}{%- endif %}
+    {%- endfor %}
+{%- endmacro %}


why do we need this?

Priority group related config is not supported on management ports, SAI throws an error. The change made in the buffers_defaults_t2.j2 is to skip programming it on those ports.
Error thrown by SAI

SAI_API_PORT:brcm_sai_set_port_attribute:3764 PG config not supported, Port:257 type is MGMT port 2025 Oct 15 22:51:03.494097 ERR syncd#syncd: :- sendApiResponse: api SAI_COMMON_API_SET failed in syncd mode: SAI_STATUS_ATTR_NOT_SUPPORTED_0 2025 Oct 15 22:51:03.494119 ERR syncd#syncd: :- processQuadEvent: VID: oid:0x1000000000041 RID: oid:0x100000101

mssonicbld · 2025-10-29T17:09:15Z

/azp run Azure.sonic-buildimage

azure-pipelines · 2025-10-29T17:09:30Z

Azure Pipelines successfully started running 1 pipeline(s).

mssonicbld · 2025-10-29T17:27:16Z

/azp run Azure.sonic-buildimage

azure-pipelines · 2025-10-29T17:27:30Z

Azure Pipelines successfully started running 1 pipeline(s).

mssonicbld · 2025-10-30T14:40:00Z

/azp run Azure.sonic-buildimage

azure-pipelines · 2025-10-30T14:40:13Z

Azure Pipelines successfully started running 1 pipeline(s).

* add device dir for nh-5010 * add sonic-platform-modules-nexthop support for nh-5010 * more nh-5010 device dir changes * remove eeprom utils change * additional nh platform support changes * macsec property and tap changes * Update adm1266 test to use pd-plugin.json from nh-5010 * use nh_powercycle for nh platforms * address review comments * update top level build files * remove spi module from pddf json * follow-up cleanup after resolving merge conflict * empty commit to retrigger checks --------- Co-authored-by: Louis Maliyam <louis@nexthop.ai>

* add device dir for nh-5010 * add sonic-platform-modules-nexthop support for nh-5010 * more nh-5010 device dir changes * remove eeprom utils change * additional nh platform support changes * macsec property and tap changes * Update adm1266 test to use pd-plugin.json from nh-5010 * use nh_powercycle for nh platforms * address review comments * update top level build files * remove spi module from pddf json * follow-up cleanup after resolving merge conflict * empty commit to retrigger checks --------- Co-authored-by: Louis Maliyam <louis@nexthop.ai> Signed-off-by: Feng Pan <fenpan@microsoft.com>

* add device dir for nh-5010 * add sonic-platform-modules-nexthop support for nh-5010 * more nh-5010 device dir changes * remove eeprom utils change * additional nh platform support changes * macsec property and tap changes * Update adm1266 test to use pd-plugin.json from nh-5010 * use nh_powercycle for nh platforms * address review comments * update top level build files * remove spi module from pddf json * follow-up cleanup after resolving merge conflict * empty commit to retrigger checks --------- Co-authored-by: Louis Maliyam <louis@nexthop.ai> Signed-off-by: xiaweijiang <xiaweijiang@microsoft.com>

roy-nexthop force-pushed the nh-5010-platform branch from 9b9392d to 8051e1e Compare October 7, 2025 20:22

eswaran-nexthop added this to SONiC Chassis Oct 7, 2025

eswaran-nexthop removed this from SONiC Chassis Oct 7, 2025

arlakshm reviewed Oct 7, 2025

View reviewed changes

device/nexthop/x86_64-nexthop_5010-r0/NH-5010-F-O32-C32/port_config.ini Outdated Show resolved Hide resolved

roy-nexthop marked this pull request as ready for review October 7, 2025 20:50

roy-nexthop requested a review from lguohan as a code owner October 7, 2025 20:50

arlakshm reviewed Oct 7, 2025

View reviewed changes

arlakshm requested review from judyjoseph, prgeor and vmittal-msft October 7, 2025 21:08

roy-nexthop force-pushed the nh-5010-platform branch from 60b4a8a to 6e9b4a3 Compare October 21, 2025 19:22

arlakshm reviewed Oct 28, 2025

View reviewed changes

roy-nexthop and others added 11 commits October 29, 2025 12:17

add device dir for nh-5010

60068b8

add sonic-platform-modules-nexthop support for nh-5010

64b2f7c

more nh-5010 device dir changes

42cb8ad

remove eeprom utils change

a82f4b9

additional nh platform support changes

518596c

macsec property and tap changes

4ab385b

Update adm1266 test to use pd-plugin.json from nh-5010

f10969c

use nh_powercycle for nh platforms

6cbc4bb

address review comments

51e611f

update top level build files

d6910b3

remove spi module from pddf json

863e6e8

roy-nexthop force-pushed the nh-5010-platform branch from 8982016 to 863e6e8 Compare October 29, 2025 17:09

follow-up cleanup after resolving merge conflict

e2e4c98

empty commit to retrigger checks

6c7f0f3

arlakshm approved these changes Oct 31, 2025

View reviewed changes

rlhui merged commit 744e74f into sonic-net:master Oct 31, 2025
21 checks passed

domingo-nexthop mentioned this pull request Mar 5, 2026

Enable kdump on Nexthop devices #25922

Merged

8 tasks

		@@ -0,0 +1,3 @@
		[Unit]
		Wants=pmon.service

		@@ -0,0 +1,3 @@
		[Unit]
		Wants=pmon.service

Conversation

roy-nexthop commented Sep 30, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Why I did it

Work item tracking

How I did it

How to verify it

Which release branch to backport (provide reason below if selected)

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

Uh oh!

mssonicbld commented Sep 30, 2025

Uh oh!

azure-pipelines bot commented Sep 30, 2025

Uh oh!

mssonicbld commented Sep 30, 2025

Uh oh!

azure-pipelines bot commented Sep 30, 2025

Uh oh!

mssonicbld commented Oct 7, 2025

Uh oh!

azure-pipelines bot commented Oct 7, 2025

Uh oh!

mssonicbld commented Oct 7, 2025

Uh oh!

azure-pipelines bot commented Oct 7, 2025

Uh oh!

mssonicbld commented Oct 7, 2025

Uh oh!

azure-pipelines bot commented Oct 7, 2025

Uh oh!

Uh oh!

eswaran-nexthop commented Oct 7, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

arlakshm Oct 7, 2025

Choose a reason for hiding this comment

Uh oh!

roy-nexthop Oct 8, 2025

Choose a reason for hiding this comment

Uh oh!

arlakshm Oct 28, 2025

Choose a reason for hiding this comment

Uh oh!

roy-nexthop Oct 29, 2025

Choose a reason for hiding this comment

Uh oh!

roy-nexthop Oct 30, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

mssonicbld commented Oct 15, 2025

Uh oh!

azure-pipelines bot commented Oct 15, 2025

Uh oh!

mssonicbld commented Oct 21, 2025

Uh oh!

azure-pipelines bot commented Oct 21, 2025

Uh oh!

mssonicbld commented Oct 21, 2025

Uh oh!

azure-pipelines bot commented Oct 21, 2025

Uh oh!

mssonicbld commented Oct 22, 2025

Uh oh!

azure-pipelines bot commented Oct 22, 2025

Uh oh!

azure-pipelines bot commented Oct 24, 2025

Uh oh!

arlakshm Oct 28, 2025

Choose a reason for hiding this comment

Uh oh!

arlakshm Oct 28, 2025

Choose a reason for hiding this comment

Uh oh!

prasoon-nexthop Oct 29, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

roy-nexthop commented Sep 30, 2025 •

edited

Loading

prasoon-nexthop Oct 29, 2025 •

edited

Loading