Skip to content

refactor(vmm): remove OCI registry image pull - #1282

Merged
kvinwang merged 1 commit into
nextfrom
fix/vmm-parser-bounds
Sep 24, 2026
Merged

kvinwang merged 1 commit into
nextfrom
fix/vmm-parser-bounds

Conversation

@kvinwang

@kvinwang kvinwang commented Sep 20, 2026 •

Copy link
Copy Markdown
Collaborator

Guest images are not published to an OCI registry and there is no plan to, so the VMM's registry client has no users. Rather than harden its manifest and blob handling (the original scope of this PR), remove it.

Removed:

  • vmm/src/app/registry.rs and the [image] registry setting
  • ListRegistryImages / PullRegistryImage RPCs
  • the registry section of the web UI Images panel (local image list/delete stays)
  • os/image/dstack-image-oci.sh and its docs

A leftover registry line in vmm.toml is ignored.

Verification: cargo test -p dstack-vmm, clippy and fmt clean, UI builds.

@kvinwang
kvinwang force-pushed the fix/vmm-parser-bounds branch from c2f6562 to 5e3c60a Compare September 24, 2026 08:32
@kvinwang kvinwang changed the title fix(vmm): a registry digest string aborts the whole VMM, and a guest names an unbounded event fix(vmm): validate registry blob digests and bound layer downloads Sep 24, 2026
Guest images are not published to an OCI registry and there is no plan to,
so the registry client in dstack-vmm had no users while still parsing
registry-supplied manifests inside the VMM process. Remove it along with the
[image] registry setting, the ListRegistryImages and PullRegistryImage RPCs,
the registry section of the Images panel, and the dstack-image-oci.sh push
script. The Images panel keeps listing and deleting local images.
@kvinwang
kvinwang force-pushed the fix/vmm-parser-bounds branch from a0821ec to 469dffb Compare September 24, 2026 12:40
@kvinwang kvinwang changed the title fix(vmm): validate registry blob digests and bound layer downloads refactor(vmm): remove OCI registry image pull Sep 24, 2026
@kvinwang
kvinwang merged commit b80f0cb into next Sep 24, 2026
16 checks passed
@kvinwang
kvinwang deleted the fix/vmm-parser-bounds branch September 24, 2026 14:25
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.

1 participant