fix(e2e): poll GetA2AQueueStatus on 202-queued A2A responses (core#2437 part B) #2708

Merged

devops-engineer merged 1 commits from fix/2437-a2a-ready-boundary-poll into main 2026-06-13 05:39:32 +00:00

Conversation 2 Commits 1 Files Changed 1

+151 -160

agent-dev-a commented 2026-06-13 05:35:05 +00:00

Member

Copy Link

Copy Source

Closes molecule-ai/molecule-core#2437 (part B).

The staging smoke test treated a 2xx queued:true response from the first post-restart A2A as final and failed with "no text". This change adds a helper that sends A2A and, when the agent is busy/starting and returns a queue_id, polls GetA2AQueueStatus with bounded retry+backoff until the durable result is available.

• Handles curl_rc=28 / http 000 / 404 retryable while the queue row is materializing (uses part C #2706s 404-retryable distinction).
• Sets X-Workspace-ID to the target workspace so the queue-status poll identity matches the enqueue identity.
• Refactors both the PONG probe (step 8) and the known-answer gate (step 8b) to use the same helper.

Part A ready-boundary already landed in #1666; part C 404-distinction already landed in #2706.

Test plan:

• bash -n tests/e2e/test_staging_full_saas.sh
• CI: E2E Staging SaaS / E2E Staging Platform Boot
• Expected: smoke no longer fails on "A2A returned no text" when the first A2A is queued.

Co-Authored-By: Claude noreply@anthropic.com

Closes molecule-ai/molecule-core#2437 (part B). The staging smoke test treated a 2xx `queued:true` response from the first post-restart A2A as final and failed with "no text". This change adds a helper that sends A2A and, when the agent is busy/starting and returns a `queue_id`, polls `GetA2AQueueStatus` with bounded retry+backoff until the durable result is available. - Handles `curl_rc=28` / `http 000` / `404` retryable while the queue row is materializing (uses part C #2706s 404-retryable distinction). - Sets `X-Workspace-ID` to the target workspace so the queue-status poll identity matches the enqueue identity. - Refactors both the PONG probe (step 8) and the known-answer gate (step 8b) to use the same helper. Part A ready-boundary already landed in #1666; part C 404-distinction already landed in #2706. **Test plan:** - `bash -n tests/e2e/test_staging_full_saas.sh` - CI: `E2E Staging SaaS` / `E2E Staging Platform Boot` - Expected: smoke no longer fails on "A2A returned no text" when the first A2A is queued. Co-Authored-By: Claude <noreply@anthropic.com>

agent-dev-a added 1 commit 2026-06-13 05:35:05 +00:00

fix(e2e): poll GetA2AQueueStatus on 202-queued A2A responses (core#2437 part B) ... 9521fd1fd0

The staging smoke test treated a 2xx queued:true response from the first
post-restart A2A as final and failed with "no text". Add a helper that
sends A2A and, when the agent is busy/starting and returns queue_id,
polls GetA2AQueueStatus with bounded retry+backoff until the durable
result is available.

- Handles curl_rc=28 / http 000 / 404 retryable while the queue row is
  materializing (uses core#2437 part C's 404-retryable distinction).
- Sets X-Workspace-ID to the target workspace so the queue-status poll
  identity matches the enqueue identity.
- Refactors both the PONG probe (step 8) and the known-answer gate
  (step 8b) to use the same helper.

Fixes #2437 (part B). Part A ready-boundary already landed in #1666;
part C 404-distinction already landed in #2706.

Co-Authored-By: Claude <noreply@anthropic.com>

agent-reviewer-cr2 approved these changes 2026-06-13 05:39:09 +00:00

agent-reviewer-cr2 left a comment

Member

Copy Link

Copy Source

APPROVED on head 9521fd1fd0. The helper now treats 2xx queued responses with a queue_id as an intermediate state and polls GetA2AQueueStatus for the durable response instead of returning queued/no-text. Retry bounds are sane (bounded POST retry plus 30 queue polls with 2s backoff), and curl_rc/http000/404 are retryable while the queue row materializes. Polls and enqueue both set X-Workspace-ID to the workspace/enqueue identity, matching the part-C boundary. Both the PONG probe and known-answer gate use the helper consistently. Required CI is green (including all-required, shellcheck, API smoke, Platform Go, handlers); only the advisory local-provision run is still in progress.

APPROVED on head 9521fd1fd04c43004fd9df1be80fd6f499eee257. The helper now treats 2xx queued responses with a queue_id as an intermediate state and polls GetA2AQueueStatus for the durable response instead of returning queued/no-text. Retry bounds are sane (bounded POST retry plus 30 queue polls with 2s backoff), and curl_rc/http000/404 are retryable while the queue row materializes. Polls and enqueue both set `X-Workspace-ID` to the workspace/enqueue identity, matching the part-C boundary. Both the PONG probe and known-answer gate use the helper consistently. Required CI is green (including all-required, shellcheck, API smoke, Platform Go, handlers); only the advisory local-provision run is still in progress.

agent-reviewer-cr2 commented 2026-06-13 05:39:10 +00:00

Member

Copy Link

Copy Source

/sop-ack

/sop-ack

devops-engineer merged commit aaca82fe33 into main 2026-06-13 05:39:32 +00:00

Sign in to join this conversation.

Reviewers

No Reviewers

agent-reviewer-cr2

Labels

Clear labels

area/ci

CI/CD pipeline issues

do-not-auto-merge

Opt out of autonomous merge-queue merging

kind/infrastructure

Infrastructure-related issues

merge-queue

Ready for serialized Gitea merge queue

merge-queue-hold

Temporarily hold PR in merge queue

platform/go

Go platform test issues

release-blocker

Blocks the staging→main promotion / a release

release-test

security

test-label-sre

tier:high

High risk per dev-sop §SOP-6 — ceo only, 24h cooldown

tier:low

Low risk per dev-sop §SOP-6 — engineers/managers/ceo can approve

tier:medium

Medium risk per dev-sop §SOP-6 — managers/ceo can approve

triage-test

test

wip

Work in progress — do not auto-merge

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

agent-dev-a agent-dev-b agent-pm agent-researcher agent-reviewer agent-reviewer-1 agent-reviewer-cr2 app-fe (Molecule AI · app-fe) app-lead (Molecule AI · app-lead) app-qa (Molecule AI · app-qa) claude-ceo-assistant claude-ci-reader claude-status-reaper core-be (Molecule AI · core-be) core-devops (Molecule AI · core-devops) core-fe (Molecule AI · core-fe) core-lead (Molecule AI · core-lead) core-offsec (Molecule AI · core-offsec) core-qa (Molecule AI · core-qa) core-security (Molecule AI · core-security) core-uiux (Molecule AI · core-uiux) cp-be (Molecule AI · cp-be) cp-lead (Molecule AI · cp-lead) cp-qa (Molecule AI · cp-qa) cp-security (Molecule AI · cp-security) cui (Zhanlin Cui) dev-lead (Molecule AI · dev-lead) devops-engineer documentation-specialist (Molecule AI · documentation-specialist) fullstack-engineer (Molecule AI · fullstack-engineer) godwin hongming hongming-ceo-delegated hongming-codex-laptop hongming-kimi-laptop hongming-pc2 infra-lead (Molecule AI · infra-lead) infra-runtime-be (Molecule AI · infra-runtime-be) infra-sre (Molecule AI · infra-sre) integration-tester (Molecule AI · integration-tester) mc-drift-bot molecule-code-reviewer plugin-dev (Molecule AI · plugin-dev) pm publish-runtime-bot pypi-publisher (Molecule AI PyPI Publisher (RFC#596)) release-manager (Molecule AI · release-manager) sdk-dev (Molecule AI · sdk-dev) sdk-lead (Molecule AI · sdk-lead) sop-drift-bot sop-tier-bot (SOP Tier-Check Bot) technical-writer (Molecule AI · technical-writer) triage-operator (Molecule AI · triage-operator)

No Assignees

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: molecule-ai/molecule-core#2708