Skip to content

fix(vm): scope rootfs cache by openshell version#1587

Merged
maxamillion merged 1 commit into
mainfrom
noissue-vm-rootfs-cache-version/an
May 27, 2026
Merged

fix(vm): scope rootfs cache by openshell version#1587
maxamillion merged 1 commit into
mainfrom
noissue-vm-rootfs-cache-version/an

Conversation

@drew
Copy link
Copy Markdown
Collaborator

@drew drew commented May 27, 2026

Summary

Scope VM bootstrap and prepared rootfs cache identities by OpenShell version so upgraded gateways build fresh guest rootfs disks instead of reusing cached disks with older embedded supervisors.

Related Issue

None.

Changes

  • Bump the VM bootstrap and prepared rootfs cache layout namespaces to v3.
  • Include openshell_core::VERSION in both VM rootfs cache identities.
  • Update cache identity unit tests and document the version-scoped cache invariant in the VM driver README.

Testing

  • mise run pre-commit passes
  • Unit tests added/updated
  • E2E tests added/updated (not applicable)

Additional validation: mise run e2e:vm was attempted. It built a fresh v3 version-scoped rootfs cache, then failed at sandbox readiness with the existing VM supervisor callback reachability timeout: Waiting for VM supervisor with repeated NET:FAIL host.openshell.internal:.

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs updated (if applicable)

Signed-off-by: Drew Newberry <anewberry@nvidia.com>
@drew drew requested review from a team, derekwaynecarr, maxamillion and mrunalp as code owners May 27, 2026 06:56
Copy link
Copy Markdown
Collaborator

@maxamillion maxamillion left a comment

Choose a reason for hiding this comment

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

+1 - I'm a fan of aggressive cache invalidation in this instance.

@maxamillion maxamillion merged commit d8010ef into main May 27, 2026
26 checks passed
@maxamillion maxamillion deleted the noissue-vm-rootfs-cache-version/an branch May 27, 2026 18:00
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