Skip to content

[Nexthop] Distro Infra: Support multiple CPU interfaces#983

Open
travisb-nexthop wants to merge 1 commit intofacebook:mainfrom
nexthop-ai:distro_infra_multi_intf
Open

[Nexthop] Distro Infra: Support multiple CPU interfaces#983
travisb-nexthop wants to merge 1 commit intofacebook:mainfrom
nexthop-ai:distro_infra_multi_intf

Conversation

@travisb-nexthop
Copy link
Contributor

Pre-submission checklist

  • I've ran the linters locally and fixed lint errors related to the files I modified in this PR. You can install the linters by running pip install -r requirements-dev.txt && pre-commit install
  • pre-commit run

Summary

If a CPU board has multiple network interfaces, it may be that the externally facing management port is not discovered as net0 by iPXE. In that case net1 will be the external port and will get an IP, but the autoexec.ipxe script will attempt to retrieve the installer using the MAC address of the non-external interface.

Use netX instead, which represents the last successful interface to be 'opened' and be assigned and IP address.

Test Plan

Successfully PXE booted fboss103 with these modifications. fboss103 started enumerating the internal interface to the BMC as net0 sometime recently. Prior to this change it would fail because it did not receive any DHCP response from the BMC.

If a CPU board has multiple network interfaces, it may be that the
externally facing management port is not discovered as net0 by iPXE. In
that case net1 will be the external port and will get an IP, but the
autoexec.ipxe script will attempt to retrieve the installer using the
MAC address of the non-external interface.

Use netX instead, which represents the last successful interface to be
'opened' and be assigned and IP address.

Successfully PXE booted fboss103 with these modifications. fboss103
started enumerating the internal interface to the BMC as net0
sometime recently..
@travisb-nexthop travisb-nexthop requested a review from a team as a code owner March 6, 2026 23:11
@meta-cla meta-cla bot added the CLA Signed label Mar 6, 2026
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.

1 participant