Skip to content
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.

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

node-installer: run nydus snapshotter on bare metal platforms #798

Merged
merged 7 commits into from
Aug 9, 2024

Conversation

katexochen
Copy link
Member

Run the nydus-snapshotter on all bare-metal platforms. This snapshotter is used to implement the Kata image guest-pull workflow, where the image is pulled directly inside the CVM. The snapshotter on the host will add the require information to requests the shim will then send to the agent. The agent will use image-rs to pull the image from a registry.

The snapshotter currently requires setting a config options of the containerd plugin. As the table header of this option is part of the k3s/RKE base template, we need to disable the templating to not add the header twice to the document. Opposed to what is written in the k3s docs, it isn't possible to just delete the template and write the config itself, so we copy the content of the rendered config into the template and modify the template onward.

The piece missing in this PR is the installation of nydus-overlayfs on the host. This is a mount helper used to create a snapshot on the host while maintaining the custom mount labels that should be delivered to the kata-agent. It is difficult to version as it isn't possible to call it by path, and it's name is relied upon in the agent. It will be installed as part of a followup PR (present on our TDX dev machine for testing purpose).

@katexochen katexochen added the feature request Proposing a new feature label Aug 9, 2024
@katexochen katexochen requested review from burgerdev and msanft and removed request for burgerdev August 9, 2024 06:54
@katexochen katexochen added feature Shiny new feature for our users and removed feature request Proposing a new feature labels Aug 9, 2024
nodeinstaller/node-installer.go Outdated Show resolved Hide resolved
packages/by-name/nydus-snapshotter/package.nix Outdated Show resolved Hide resolved
@katexochen katexochen merged commit ccce725 into main Aug 9, 2024
9 checks passed
@katexochen katexochen deleted the p/nydus-1 branch August 9, 2024 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Shiny new feature for our users
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants