Conversation
Advance the public sandboxd pin to the merged runtime bundle integration and consume its shared checksum-verifying installer. Preserve runsc, the containerd shim, and all adjacent helpers in the final node image so checkpoint and restore use one matching release installation. Extend the CI payload checks and deployment guidance to cover the complete bundle. Keep YuanRong and the legacy actor SDK versions unchanged. Signed-off-by: Tianyu Zhou <albert.zty@antgroup.com>
|
Non-blocking documentation consistency note: this sandboxd gitlink also includes inclusionAI/sandboxd#57. Bundled standalone, Helm, and Terraform configurations already enable Firecracker virtio-fs, so lower-level callers can now explicitly request writable host-directory binds, although existing mounts and the Python SDK Could we update |
rcyw
left a comment
There was a problem hiding this comment.
The complete gVisor bundle is checksum-verified, installed as one version-matched set, and covered by final-image checks. CI is green. The writable virtio-fs documentation follow-up is non-blocking and recorded in the PR discussion.
Summary
Advance sandboxd from
f7ab6d4ato public maine67b2585, which includes inclusionAI/sandboxd#60, and consume the promoted gVisorrelease-20260817.0-akernel.3archive through sandboxd's checksum-verifying installer. Copy runsc, the containerd shim, and all four adjacentgvisor-bin/helpers into the final node image rather than installing only runsc.Extend CI's final-image checks to require every helper and document the complete-bundle contract. Runtime release URLs and checksums remain owned by the sandboxd manifest; there are no AKernel-specific overrides or runtime downloads of missing helpers.
The gitlink also brings the already-merged sandboxd changes for additional runsc arguments (#58), writable Firecracker virtio-fs directory mounts (#57), and shared ChunkDB capacity (#59). This PR does not change deployment defaults to opt into those features. YuanRong core/RRT remain
0.10.2rc6, the default SDK backend remains0.10.2rc9, and the legacy actor SDK remains pinned to0.9.9. Other component pins are unchanged.Validation
All manual compilation and runtime validation took place on the dedicated test node, not the local workstation. The public runsc-only two-Dockerfile build uses the published gVisor URL and digest, not the former candidate mirror. The complete sandboxd runtime matrix already passed in the merged upstream PR.
sha256:22212a310f3aee6af3cdddb449a11010776f386f536357808b6d2f816205cb7c. All six gVisor executables are executable and byte-for-byte identical to the promoted candidate bundle.make sdk-check: 273 unit tests, Ruff and mypy passed.make deploy-script-check,make versions, andgit diff --checkpassed.Standalone testing used a separate native-XFS data directory, container names, and cgroup root on the existing test host; private registry credentials remain outside the PR. The manual node build follows the public CI runsc-only profile and does not claim to rebuild or revalidate GPU/Kata/Firecracker payloads in AKernel.
Known independent limitation
Extended concurrent creation/reload testing previously exposed a YuanRong stop/recovery lifecycle race (
instance exist but not the same one, 80025) on this public AKernel baseline. The stale retry/response sequence was also seen with the prior runsc release in a separate ordinary-sandbox control test. YuanRong maintainers are handling that issue separately. This dependency update does not claim to fix it or to have passed the extended mixed-pressure workload. No production cluster, internal ACI configuration, or internal branch is changed.