Skip to content

agentic(sibling): write result JSON to /workspace so the result guard finds it#2165

Merged
ichbinblau merged 1 commit into
SemiAnalysisAI:amd/agentx-v1.0-yanfeifrom
AMD-yanfeiwang:yanfwang/dsv4-sibling-output-fix
Jul 11, 2026
Merged

agentic(sibling): write result JSON to /workspace so the result guard finds it#2165
ichbinblau merged 1 commit into
SemiAnalysisAI:amd/agentx-v1.0-yanfeifrom
AMD-yanfeiwang:yanfwang/dsv4-sibling-output-fix

Conversation

@AMD-yanfeiwang

Copy link
Copy Markdown

Summary

Fixes the DSv4 agentic sibling-client run failing at the end with
Run failed: expected 1 agentic results, found 0. (e.g. run 29095134929, dsv4 ... c32).

Root cause: the node-0 sibling client set AGENTIC_OUTPUT_DIR=/run_logs/slurm_job-* (host /tmp), so process_agentic_result.py saved the aggregated ${RESULT_FILENAME}_conc<N>.json outside GITHUB_WORKSPACE. The workflow guard globs ${RESULT_FILENAME}_conc*.json in GITHUB_WORKSPACE and found 0.

The co-located path leaves AGENTIC_OUTPUT_DIR unset, so it defaults to INFMAX_CONTAINER_WORKSPACE=/workspace — which is the host repo bind-mount (== GITHUB_WORKSPACE). Dropping the override makes the sibling match: raw artifacts still go under the trace_replay log dir (/run_logs), only the top-level result JSON moves to /workspace where the guard and upload steps expect it. /workspace is bind-mounted writable in the sibling docker run, same as the co-located server container.

Changes

  • benchmarks/multi_node/amd_utils/server_sglang.sh: remove the AGENTIC_OUTPUT_DIR=/run_logs/... line from the sibling client.env (single line + explanatory comment).

Test plan

  • Re-dispatch dsv4-fp4-mi355x-sglang-disagg-agentic-hicache; confirm Saved aggregated agentic result to /workspace/${RESULT_FILENAME}_conc<N>.json and the result-count guard passes.
  • Confirm artifacts (LOGS/agentic/**) still upload.

The node-0 sibling client pinned AGENTIC_OUTPUT_DIR=/run_logs/slurm_job-* (host
/tmp), so the aggregated ${RESULT_FILENAME}_conc<N>.json landed outside
GITHUB_WORKSPACE and the workflow result-count guard failed with
"expected 1 agentic results, found 0" (run 29095134929, dsv4 c32).

The co-located path leaves AGENTIC_OUTPUT_DIR unset, defaulting to
INFMAX_CONTAINER_WORKSPACE=/workspace (host repo bind-mount == GITHUB_WORKSPACE).
Drop the override so the sibling matches: raw artifacts still go under the
trace_replay log dir (/run_logs), only the top-level result JSON moves to
/workspace where the guard/upload steps expect it.

@claude claude Bot 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.

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

@ichbinblau ichbinblau merged commit b0b4b04 into SemiAnalysisAI:amd/agentx-v1.0-yanfei Jul 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants