Skip to content

Let general-purpose instances start and route on a VPC network - #531

Merged
scotwells merged 2 commits into
mainfrom
fix/cnitap-default-ifname
Sep 12, 2026
Merged

scotwells merged 2 commits into
mainfrom
fix/cnitap-default-ifname

Conversation

@scotwells

Copy link
Copy Markdown
Contributor

Instances on the general-purpose runtime class could not start on a VPC network. Every sandbox was rejected and retried forever, so the instance never reached a running state, and the two defects behind that had to be cleared in order: the instance could not start, and once it started it had no route off its own link.

The network attachment now describes the instance's own interface under the name the runtime asks for, which is how a container runtime finds the instance's address and admits the sandbox. The device itself still lives in the host namespace, which is what lets a microVM guest attach to it. The default route handed to the guest is now marked on-link, so the guest kernel accepts it: a VPC gateway sits outside the prefix allocated to an attachment, and without that marking the guest comes up addressed but unable to reach anything.

Verified against the Kata release the cells run, which honours the on-link marking end to end. Unikernel instances and container workloads on the veth path are unaffected.

This unblocks the next stage of general-purpose VPC attachment rather than completing it: an instance has not yet been observed reaching the network end to end.

Follows #528. Related: datum-labs/kata-provider#13, datum-cloud/cloud#20.

Separately, the pod admission path writes the same network attachment as both a secondary and the default network, so the attachment is set up twice per sandbox. That belongs in datum-cloud/cloud and is not addressed here.

🤖 Generated with Claude Code

scotwells and others added 2 commits September 12, 2026 15:27
A general-purpose instance attached to a VPC network never reached a
running state: the container runtime rejected every sandbox because the
network result did not describe the instance's own interface under the
name the runtime asked for, so it could not find an address for it.

The tap attachment now reports that interface under the requested name.
The device still lives in the host namespace, which is what lets a
microVM guest attach to it, and CHECK resolves the device from the
attachment rather than from the reported name.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
A guest that came up on a VPC network could not reach anything off-link:
its kernel refused the default route, because a VPC gateway sits outside
the prefix allocated to the attachment and nothing said the gateway was
reachable on the link.

The route handed to the guest is now marked on-link, the same treatment
the host side of the attachment already gets.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@scotwells
scotwells requested a review from a team as a code owner September 12, 2026 20:27
@scotwells
scotwells requested review from ecv and privateip September 12, 2026 20:27
@scotwells
scotwells merged commit 5b69d71 into main Sep 12, 2026
10 checks passed
@scotwells
scotwells deleted the fix/cnitap-default-ifname branch September 12, 2026 20:30
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.

2 participants