Skip to content

Add end-to-end coverage for large enclave responses - #8174

Open
Amaury Chamayou (achamayou) with Copilot wants to merge 11 commits into
mainfrom
copilot/add-end-to-end-test-large-responses
Open

Add end-to-end coverage for large enclave responses#8174
Amaury Chamayou (achamayou) with Copilot wants to merge 11 commits into
mainfrom
copilot/add-end-to-end-test-large-responses

Conversation

Copilot AI commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Existing stress tests cover large requests and KV writes, but not enclave-generated responses near or above ringbuffer capacity.

Changes

  • Add an authenticated, read-only, non-forwarded response generator.
  • Read memory.circuit_size and memory.max_msg_size from the active primary's rendered configuration.
  • Derive a safe maximum generated response as half of memory.max_msg_size, preserving headroom for QuickJS and response serialization.
  • Configure the generator through a member-authenticated endpoint, then verify responses at the configured ringbuffer capacity and derived response cap.
  • Verify each response's status, exact size, and boundary bytes before existing KV stress tests run.

Closes #5039

Co-authored-by: achamayou <4016369+achamayou@users.noreply.github.com>
Copilot AI changed the title [WIP] Add end-to-end test for large HTTP responses Add end-to-end coverage for large enclave responses Aug 18, 2026
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@achamayou
Amaury Chamayou (achamayou) requested a balanced review from Copilot August 19, 2026 19:05
@achamayou
Amaury Chamayou (achamayou) marked this pull request as ready for review August 19, 2026 19:06

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Warning

Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.

Adds end-to-end coverage for enclave-generated HTTP responses at/above the ringbuffer capacity, to complement existing KV stress tests and close #5039.

Changes:

  • Added a new readonly, authenticated /batch/generate endpoint that returns a response body of requested size.
  • Added an e2e test that requests 16 MiB and 32 MiB responses and verifies status, size, and boundary bytes before existing stress tests.
  • Updated the “run to destruction” entrypoint logic to adjust max_msg_size_bytes around the new large-response coverage.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
tests/e2e_batched.py Adds large-response e2e test and updates max message size configuration around destruction runs.
src/apps/batched/src/batched.js Adds response generator handler that returns a body of requested size.
src/apps/batched/app.json Registers the new /batch/generate endpoint with authn/readonly/no-forwarding settings.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread tests/e2e_batched.py Outdated
Comment thread src/apps/batched/src/batched.js
Comment thread tests/e2e_batched.py Outdated
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
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.

Add an end-to-end test that amplifies the response size

3 participants