Skip to content

Conversation

shmuelk
Copy link
Collaborator

@shmuelk shmuelk commented Sep 29, 2025

Newer versions of Kind use containerd under the covers in the simulated K8S nodes.

There is a known incompatibility between Docker and containerd with respect to multi-platform images. Docker doesn't pull al of the layers of such images. It only pulls the layers that are applicable to the platform (i.e. AMD64 or ARM64) in question. Containerd then issues an error about missing layers.

See kubernetes-sigs/kind#3795 for more details.

This PR replaces the command kind load docker-image... with docker save and kind load image-archive. The docker save is done specifying the local platform, to only save for the appropriate platform.

elevran
elevran previously approved these changes Oct 20, 2025
Copy link
Collaborator

@elevran elevran left a comment

Choose a reason for hiding this comment

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

@shmuelk please rebase

/lgtm
/approve

Signed-off-by: Shmuel Kallner <[email protected]>
@elevran
Copy link
Collaborator

elevran commented Oct 22, 2025

/lgtm
/approve

@github-actions github-actions bot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Oct 22, 2025
@github-actions github-actions bot merged commit 8d73dc4 into llm-d:main Oct 22, 2025
6 checks passed
@github-project-automation github-project-automation bot moved this from In review to Done in llm-d-inference-scheduler Oct 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm "Looks good to me", indicates that a PR is ready to be merged.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants