[PR #233] [CLOSED] test(e2e): cover multi-node p2p across sandbox churn #273

New issue

Closed

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

bsakaguchi commented

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

Owner

📋 Pull Request Information

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

Base: main ← Head: test/p2p-multinode-integration

📝 Commits (1)

95519c1 test(e2e): cover multi-node p2p across sandbox churn

📊 Changes

4 files changed (+328 additions, -2 deletions)

View changed files

📝 .github/workflows/integration-tests.yml (+3 -0)
📝 Makefile (+18 -1)
📝 scripts/tests/e2e/run_e2e.sh (+3 -1)
➕ scripts/tests/e2e/suites/15_p2p_multinode.sh (+304 -0)

📄 Description

What

Add a dedicated multi-node P2P compose E2E suite.
Start one Firecracker sandbox on each AgentENV node.
Verify sandbox veth creation and cleanup are observed.
Exercise P2P layer transfers in both directions before and after sandbox deletion.
Disable the origin path to prove that transfers use P2P rather than HTTP fallback.
Verify HTTP 206 responses, content size, and SHA256 integrity.
Add a dedicated Make target and CI workflow step.
Make the E2E template build timeout configurable.

Validation

Multi-node P2P E2E: 33 assertions passed
E2E summary: 1 passed, 0 failed
Shell syntax checks passed
make fmt 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/233 **Author:** [@WindSongShaoGe](https://github.com/WindSongShaoGe) **Created:** 8/30/2026 **Status:** ❌ Closed **Base:** `main` ← **Head:** `test/p2p-multinode-integration` --- ### 📝 Commits (1) - [`95519c1`](https://github.com/kvcache-ai/AgentENV/commit/95519c14e64f229f1121a2e8e6afd07956c12b70) test(e2e): cover multi-node p2p across sandbox churn ### 📊 Changes **4 files changed** (+328 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/integration-tests.yml` (+3 -0) 📝 `Makefile` (+18 -1) 📝 `scripts/tests/e2e/run_e2e.sh` (+3 -1) ➕ `scripts/tests/e2e/suites/15_p2p_multinode.sh` (+304 -0) </details> ### 📄 Description ## What - Add a dedicated multi-node P2P compose E2E suite. - Start one Firecracker sandbox on each AgentENV node. - Verify sandbox veth creation and cleanup are observed. - Exercise P2P layer transfers in both directions before and after sandbox deletion. - Disable the origin path to prove that transfers use P2P rather than HTTP fallback. - Verify HTTP 206 responses, content size, and SHA256 integrity. - Add a dedicated Make target and CI workflow step. - Make the E2E template build timeout configurable. ## Validation - Multi-node P2P E2E: 33 assertions passed - E2E summary: 1 passed, 0 failed - Shell syntax checks passed - `make fmt` 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:43 +00:00