[PR #172] [CLOSED] fix(overlaybd): probe p2p before remote runtime initialization #226

New issue

Closed

opened 2026-10-29 14:21:13 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-29 14:21:13 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/kvcache-ai/AgentENV/pull/172
Author: @WindSongShaoGe
Created: 8/14/2026
Status: ❌ Closed

Base: main ← Head: fix/p2p-probe-before-remote-runtime

📝 Commits (1)

8d8fa6a fix(overlaybd): probe p2p before remote runtime initialization

📊 Changes

2 files changed (+132 additions, -12 deletions)

View changed files

📝 storage/overlaybd/src/image/image_file.rs (+45 -1)
📝 storage/overlaybd/src/image/image_service.rs (+87 -11)

📄 Description

Summary

Probe the local P2P facade before initializing the shared remote runtime.
Prevent the first remote layer reads from initializing the runtime without P2P acceleration.
Use an unaccelerated registry path for P2P UUID probing to avoid recursively routing the probe through the P2P HTTP facade.
Add regression coverage for remote runtime initialization order.

Problem

The remote runtime is stored in a OnceCell. Previously, P2P acceleration was only probed from create_image_file. If another call initialized the remote runtime first, the runtime permanently captured an empty acceleration address, so subsequent layer reads bypassed P2P acceleration.

Validation

OverlayBD library tests: 310 passed, 0 failed, 1 ignored
Clippy passed with warnings denied
Rust formatting check passed
git diff --check passed

Refs #138

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/kvcache-ai/AgentENV/pull/172 **Author:** [@WindSongShaoGe](https://github.com/WindSongShaoGe) **Created:** 8/14/2026 **Status:** ❌ Closed **Base:** `main` ← **Head:** `fix/p2p-probe-before-remote-runtime` --- ### 📝 Commits (1) - [`8d8fa6a`](https://github.com/kvcache-ai/AgentENV/commit/8d8fa6ad8863a72243ad4184cea9919583371b47) fix(overlaybd): probe p2p before remote runtime initialization ### 📊 Changes **2 files changed** (+132 additions, -12 deletions) <details> <summary>View changed files</summary> 📝 `storage/overlaybd/src/image/image_file.rs` (+45 -1) 📝 `storage/overlaybd/src/image/image_service.rs` (+87 -11) </details> ### 📄 Description ## Summary - Probe the local P2P facade before initializing the shared remote runtime. - Prevent the first remote layer reads from initializing the runtime without P2P acceleration. - Use an unaccelerated registry path for P2P UUID probing to avoid recursively routing the probe through the P2P HTTP facade. - Add regression coverage for remote runtime initialization order. ## Problem The remote runtime is stored in a `OnceCell`. Previously, P2P acceleration was only probed from `create_image_file`. If another call initialized the remote runtime first, the runtime permanently captured an empty acceleration address, so subsequent layer reads bypassed P2P acceleration. ## Validation - OverlayBD library tests: 310 passed, 0 failed, 1 ignored - Clippy passed with warnings denied - Rust formatting check passed - `git diff --check` passed Refs #138 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

2026-10-29 14:21:13 +00:00