Repository navigation
build(runtime): refresh ADX rc1 and sandboxd Start fixes - #100
Conversation
Consume the matching platform and Execd archives built from Agent DX master af7d82f, including persistent local checkpoint caching. Keep the immutable OBS URLs and SHA-256 pins aligned with the ADX source gitlink. Signed-off-by: robbluo <luo1442@gmail.com>
Use sandboxd PR inclusionAI#75 for cancellable OCI preparation and typed internal timeouts with precise operation stages. Preserve registry rejection categories through the backend adapter. Refs inclusionAI#96. Signed-off-by: robbluo <luo1442@gmail.com>
|
@WenYuLuo Completed the read-only review at Independent checks completed in this review:
Merge order: merge sandboxd #75 first; then verify/update Keep #96 open until acceptance on the fixed merged build. Next, use an exclusive node with audited running binary/rootfs hashes for deadline-only with zero DELETEs, expiry followed by one DELETE, DELETE during creation, and successful cold import; verify phase/retry semantics, no late revival/duplicate allocation, and eventual independent physical cleanup without extra DELETEs or service restart. Logical deletion and physical reclamation remain separate; this does not require synchronous deletion or a new public cleanup API. This review did not build/deploy the full image or rerun opt-in native-runc fault, large-image, KVM/QEMU/macOS guest, or restart/rollback-failure acceptance. The author's deployment campaign is separate evidence. Default AKernel CI disables runc, so Standalone E2E cannot substitute for those checks. Guest reuse still requires authentication, trusted host keys and a confirmed serial-use window. This is a Conversation review recommendation, not a formal GitHub approval. |
Refresh the pinned ADX platform/Execd pair to Buildkite #162, source
af7d82f04a52d4604ea7ca4c550f09c6114dee4d, and pin sandboxd to PR #75 at02386fea2050c4a77cfaa9df5edfe72fd35fc987.The sandboxd update propagates Start cancellation through blocked OCI preparation, retains image-pull/runtime-start timeout types and stages, and preserves non-timeout registry error categories. The ADX #162 artifacts contain the corresponding ADX !51 propagation changes, persistent local checkpoint LRU reuse for S3-backed recovery, and scheduling-queue runtime requirements.
developwas refreshed at9f24718638131b09ad03a274b9ba52ff76396a38; the existing release-pin commit was already based on it, so rebase required no history rewrite. Commit9cd50bd3aa4ddff80fb1c6e13779bc2d4cb1fde4adds the sandboxd gitlink update. The Dockerfiles keep matching immutable OBS URLs and compressed SHA-256 pins for the ADX platform and Execd pair.Validation:
akernel-local/pr100-sandboxd75:9cd50bd-20261016, image IDsha256:30a02448b384688707a6cfefa3b10c6397bb94427639f3e10e4f4a8890bbe216. The fixed ADX archive and all 31 manifest-listed files pass checksum verification. All 12 installed release files, four running component binaries and runtime rootfs are checked; SDK and native-fault fixtures have identical production binary/rootfs hashes. Runtime Execd SHA-256 isd371c08efc2e3ccaf3b16ed36901b1c3374025e4e513527b722f5d8cdbe8ce0d.make sdk-check deploy-script-check deploy-standalone-test build-helper-testpasses, including SDK tests, Ruff and mypy.504 / DEADLINE_EXCEEDED / never / terminal. Same-request and authoritative failed-record replay preserve the exact message. Registry 401/403/404/429 preserve their corresponding error category and original message in both JSON and SSE; backend 429 is terminal instead of entering admission retry. Healthy create/delete succeeds before and after injection.9cd50bd, including Standalone E2E. PR publication is skipped as intended. Both task-owned local deployments are removed and the pre-existing development node remains running.DELETE submitted during the blocked create still waits behind that serialized create (32.847 seconds); DELETE after expiry returns in about 0.004 seconds. This dependency update retains lifecycle queue ordering.
The native cleanup fixture disables idle netns caching so independent physical-allocation checks can require an empty inventory. The SDK acceptance uses the normal standalone configuration. Sanitized local evidence is retained under
out/pr100-sandboxd75-20261016/evidence/; development-host results are under/root/akernel-pr100-sandboxd75-20261016/.Acceptance scope is Linux/amd64 standalone with runsc and native runc, using a complete new image built from this PR. Runtime fault fixtures use a metadata-only derived image and a rootfs extracted from this build; production binaries are unchanged. The fixed platform archive and every manifest file are verified independently, then the installed bin/sdk/runtime files and actual running binaries are checked against the artifacts. The runtime Execd matches that same platform manifest. The task-owned fixtures are isolated from the existing development node.
Refs #96. Keep the issue open for remaining integration and acceptance work.