Skip to content

feat: HostInband segment support for DPU-less provisioning#1978

Open
s3rj1k wants to merge 1 commit into
NVIDIA:mainfrom
s3rj1k:feat/host-inband-networking
Open

feat: HostInband segment support for DPU-less provisioning#1978
s3rj1k wants to merge 1 commit into
NVIDIA:mainfrom
s3rj1k:feat/host-inband-networking

Conversation

@s3rj1k
Copy link
Copy Markdown

@s3rj1k s3rj1k commented May 28, 2026

Description

Add HostInband as a first-class provisioning network: site-explorer includes HostInband segments in its scan set and tolerates a BMC interface preallocated on one (same machine_id), and admin-cli gains a host-inband-segment mode that creates a flat VPC + HostInband segment via new RPCs.

Type of Change

  • Add - New feature or capability
  • Change - Changes in existing functionality
  • Fix - Bug fixes
  • Remove - Removed features or deprecated functionality
  • Internal - Internal changes (refactoring, tests, docs, etc.)

Related Issues (Optional)

Breaking Changes

  • This PR contains breaking changes

Testing

  • Unit tests added/updated
  • Integration tests added/updated
  • Manual testing performed
  • No testing required (docs, internal refactor, etc.)

Additional Notes

@s3rj1k s3rj1k requested a review from a team as a code owner May 28, 2026 08:10
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented May 28, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

Add HostInband as a first-class provisioning network: site-explorer
includes HostInband segments in its scan set and tolerates a BMC
interface preallocated on one (same machine_id), and admin-cli gains a
host-inband-segment mode that creates a flat VPC + HostInband segment
via new RPCs.

Signed-off-by: s3rj1k <evasive.gyron@gmail.com>
@s3rj1k s3rj1k force-pushed the feat/host-inband-networking branch from 91d4286 to 9c0f89f Compare May 28, 2026 10:08
@ajf
Copy link
Copy Markdown
Collaborator

ajf commented May 28, 2026

This is giving the ability for the admin-cli dev-env command to create a Flat VPC and a host-inband segment type to do testing in dev-env?

@s3rj1k
Copy link
Copy Markdown
Author

s3rj1k commented May 28, 2026

This is giving the ability for the admin-cli dev-env command to create a Flat VPC and a host-inband segment type to do testing in dev-env?

Yes, flat network setup for devenv, when BMCs and nodes are on same subnet, still uses dhcp-relay though.

@ajf
Copy link
Copy Markdown
Collaborator

ajf commented May 28, 2026

/ok to test 9c0f89f

let mut txn = self.txn_begin().await?;

let underlay_segments =
// BMCs that share a host network end up preallocated on a HostInband
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know whether this mode is supported by other code-path or whether Inband and BMC need to be on separate networks.

@chet might know. Since no-dpu support is still work in progress, it's probably not documented yet (#1531 )

continue;
}

let ns_id: NetworkSegmentId = uuid::Uuid::new_v4().into();
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These are covered by different roles:

  • The inband segment always needs to be created by the site admin and is created only once
  • The "flat VPC" will be created by each tenant that wants to use it.

As a result of this, the create_inband_segment should not be here, and the API might need to error out.

@chet for more input how flat VPCs are supposed to be created.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants