Skip to content

[GLM-5.3 GB300] add Dynamo-SGLang AgentX agg and disagg recipes with power / [GLM-5.3 GB300] 新增 Dynamo-SGLang AgentX 聚合与分离部署配方并采集功耗 - #4706

Open
edwingao28 wants to merge 8 commits into
mainfrom
fix/glm52-gb300-dynamo-power
Open

edwingao28 wants to merge 8 commits into
mainfrom
fix/glm52-gb300-dynamo-power

Conversation

@edwingao28

@edwingao28 edwingao28 commented Nov 1, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Add GLM-5.3 NVFP4 GB300 Dynamo-SGLang AgentX: two aggregated points (TP4 and TP8 at conc 1) and six disaggregated PP4-prefill points (conc 30 to 264). The recipes port GLM-5.2 GB300, whose FP4 weights left the cluster, with NEXTN K=3 MTP and pinned weights. A new GB300 power rule routes these points to AgentX power.

Testing: sweep 41407333903 at 26d2411: 8/8 points strict power, 8/8 gsm8k 0.973 to 0.977, at most 0.7% request errors. Against GLM-5.2 GB300, output throughput per GPU is within 4% and interactivity up to 13% lower.

AI model disclosure
  • Model/version: claude-opus-5-5 (Claude Code).
  • Role: recipe port, launch-policy change, sweep verification, and PR preparation.
PR checklist and change type

Type of Change

  • Bug fix
  • New feature
  • Configuration change
  • Documentation update
  • Other (please describe)

Checklist

  • I have completed the AI model disclosure and kept it current
  • I have tested my changes locally
  • I have updated documentation if necessary
  • If python -m infx.matrix.changed --base origin/main lists affected config keys, I have appended one new entry covering them to the physical end of inferencex-e2e/perf-changelog.yaml; if it lists none, I have added none. I have not edited historical entries
  • If this PR appends an inferencex-e2e/perf-changelog.yaml entry, it carries exactly one primary sweep label (a maintainer applies it on fork PRs): full-sweep-fail-fast (recommended), full-sweep-enabled, or non-canary-full-sweep-enabled. Optional modifiers all-evals, evals-only, and agentx-fast require a primary label; the last two block reuse while applied.
  • Before merging via reuse, an authorized maintainer (OWNER/MEMBER/COLLABORATOR) has commented /use <run_id> (or the legacy /reuse-sweep-run) on this PR. Do this only once there is a final full sweep that is all green with evals passing, since after this comment the primary sweep label will no longer automatically kick off new sweeps. Remove and re-add the primary sweep label to force a new sweep.
中文

新增 GLM-5.3 NVFP4 GB300 Dynamo-SGLang AgentX 配置:2 个聚合部署点(TP4 和 TP8,并发 1)和 6 个 PP4 prefill 分离部署点(并发 30 到 264)。配方移植自 GLM-5.2 GB300(其 FP4 权重已从集群移除),改用 NEXTN K=3 MTP,并固定权重版本。新增的 GB300 功耗规则将这些点路由到 AgentX 功耗采集。

测试: sweep 41407333903(26d2411f3):8/8 个点通过严格功耗校验,8/8 个 gsm8k 评估得分 0.973 到 0.977,请求错误率最高 0.7%。与 GLM-5.2 GB300 相比,每 GPU 输出吞吐量相差不超过 4%,interactivity 最多低 13%。

AI 模型:claude-opus-5-5(Claude Code),负责配方移植、launch 策略改动、sweep 验证和 PR 准备。

中文:采集 GLM-5.2 GB300 Dynamo-SGLang AgentX 功耗
@github-actions

github-actions Bot commented Nov 1, 2026

Copy link
Copy Markdown
Contributor

Thanks for the contribution!

  • Review: If this PR changes files owned by someone other than a repository admin or @SemiAnalysisAI/core, ask one eligible CODEOWNER to complete the latest PR_REVIEW_CHECKLIST.md before contacting a core maintainer on Slack. Follow the template exactly, including As a PR reviewer and CODEOWNER, I have reviewed this and have, so sign-off verification triggers.
  • PR verification: Sweeps run only when the PR appends an inferencex-e2e/perf-changelog.yaml entry and carries exactly one primary label: full-sweep-fail-fast (strongly recommended; canary plus per-matrix fail-fast), full-sweep-enabled (canary; matrix jobs continue after a failure), or non-canary-full-sweep-enabled (no canary or fail-fast). The modifiers all-evals, evals-only, and agentx-fast require a primary label. On fork PRs, a maintainer applies the label. See sweep labels and reuse.
  • After merging: PR authors must ensure all GitHub Actions jobs pass. Transient failures often pass on rerun; see how to rerun failed jobs.
中文

感谢你的贡献!

  • **审阅:**如果 PR 修改的文件归属于仓库管理员及 @SemiAnalysisAI/core 之外的 CODEOWNER,请先联系一位有资格的 CODEOWNER 填写最新的 PR_REVIEW_CHECKLIST.md,再通过 Slack 联系核心维护者。必须严格遵循模板,并保留 As a PR reviewer and CODEOWNER, I have reviewed this and have,才能触发签核验证。
  • **PR 验证:**只有当 PR 在 inferencex-e2e/perf-changelog.yaml 末尾追加了条目,并且恰好带有一个主标签时,才会运行扫描:full-sweep-fail-fast(强烈推荐;canary 加逐矩阵 fail-fast)、full-sweep-enabled(有 canary;矩阵任务在失败后继续运行)或 non-canary-full-sweep-enabled(无 canary,也无 fail-fast)。修饰标签 all-evals、evals-only 和 agentx-fast 必须与主标签一起使用。fork PR 的标签由维护者添加。参见扫描标签与复用。
  • **合并后:**PR 作者必须确保所有 GitHub Actions 任务通过。临时性失败通常可以通过重新运行恢复;参见重新运行失败任务的说明。

@github-actions

github-actions Bot commented Nov 1, 2026

Copy link
Copy Markdown
Contributor

inferencex-e2e/infx diff: +6 -1

Generally most InferenceX submissions don't need to add changes to the infx package. On the off chance they do, generally the diff is small. Please prompt your agent to clean your PR to keep your diff to only necessary & minimal changes too. We have an increased code quality bar for changes to the infx package.

中文

大多数 InferenceX 提交通常不需要修改 infx 包。即使确实需要,改动通常也很小。请让你的 agent 清理 PR,使改动只包含必要且最小的内容。infx 包的改动有更高的代码质量要求。

@edwingao28
edwingao28 force-pushed the fix/glm52-gb300-dynamo-power branch from 1238f20 to 1a66b16 Compare November 1, 2026 09:13

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

Besides the inline finding, I checked the new GB300 GLM-5.2 PowerRule in infx/launch/drivers/srt/power.py. Its Match arguments are in the right order (prefixes, precisions, frameworks, then agentic=True). The rule also matches the existing GB200 GLM-5.2 rule exactly, which doesn't set require=True either, so leaving that flag off follows the existing pattern.

Extended reasoning...

I compared the new rule with Match's field order in infx/launch/policy.py and with the GB200 srt-multi lane, which already has an identical GLM-5.2 FP4 dynamo-sglang AgentX rule that doesn't set require.

Comment thread inferencex-e2e/perf-changelog.yaml Outdated
description:
- "Collect GPU-board power for the two aggregated and six disaggregated GLM-5.2 GB300 Dynamo-SGLang AgentX points, which currently publish no power."
- "Route GLM-5.2 FP4 AgentX dynamo-sglang on the GB300 srt-multi lane to AgentX power instead of DCGM-only power."
pr-link: https://github.com/SemiAnalysisAI/InferenceX/pull/PENDING

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.

🔴 The changelog check will reject this PR, and merge prep cannot canonicalize the link, because pull/PENDING is neither an accepted placeholder nor a PR number. validate_added_pr_link only accepts links in PR_LINK_PLACEHOLDERS (XXX or .../pull/XXX) or the exact .../pull/<pr_number> URL. Fix: set the link to https://github.com/SemiAnalysisAI/InferenceX/pull/XXX or the real PR URL. XXX is what the merge tooling rewrites.

Why this was flagged

The new tail entry at inferencex-e2e/perf-changelog.yaml:11779 sets pr-link: https://github.com/SemiAnalysisAI/InferenceX/pull/PENDING. On PR runs, validate_added_pr_link (infx/workflows/validate_perf_changelog.py:105-116) accepts only a link in PR_LINK_PLACEHOLDERS (lines 21-24: XXX and https://github.com/SemiAnalysisAI/InferenceX/pull/XXX) or the exact https://github.com/SemiAnalysisAI/InferenceX/pull/{pr_number}. Any other value raises "new PR entry must use ... or an XXX placeholder". Without a PR number, CANONICAL_PR_LINK requires \d+, so the check fails there too. prepare_perf_changelog_merge.py:108 also rejects links outside PR_LINK_PLACEHOLDERS with "unexpected pr-link". As a result the changelog validation job fails and the reuse/merge path cannot rewrite the link. No other entry in the file uses PENDING.

Verification: Trigger: an authorized maintainer runs the reuse merge (/use <run_id>) for this PR. validate_added_pr_link (validate_perf_changelog.py:112-116) raises new PR entry must use ... or an XXX placeholder for pull/PENDING, unless the value is exactly .../pull/{pr_number}. Result: the reuse merge errors out and the placeholder is never rewritten to the real PR URL.

@edwingao28 edwingao28 added the full-sweep-fail-fast Full sweep with canary gate; first failure cancels the rest of that matrix (recommended) label Nov 1, 2026
@github-actions

github-actions Bot commented Nov 1, 2026 •

Copy link
Copy Markdown
Contributor

中文:将 main 合并到 GLM-5.3 GB300 Dynamo-SGLang AgentX
中文:新增 GLM-5.3 FP4 GB300 Dynamo-SGLang AgentX 聚合配置
中文:新增 GLM-5.3 FP4 GB300 Dynamo-SGLang AgentX 分离式配置
中文:将 main 合并到 GLM-5.3 GB300 Dynamo-SGLang AgentX
@edwingao28 edwingao28 changed the title [PowerX] collect GLM-5.2 GB300 Dynamo-SGLang AgentX power / [PowerX] 采集 GLM-5.2 GB300 Dynamo-SGLang AgentX 功耗 [GLM-5.3 GB300] add Dynamo-SGLang AgentX agg and disagg recipes with power / [GLM-5.3 GB300] 新增 Dynamo-SGLang AgentX 聚合与分离部署配方并采集功耗 Nov 2, 2026
@edwingao28

Copy link
Copy Markdown
Collaborator Author

/use 41407333903

@github-actions

github-actions Bot commented Nov 2, 2026 •

Copy link
Copy Markdown
Contributor

@edwingao28 staged run 41407333903.

This run remains available across future /stage-results requests. Staging the same run ID again updates its staged data. Staging workflow

中文:更正 GLM-5.3 GB300 changelog 中的功耗规则描述。
中文:将 main 合并到 GLM-5.3 GB300 Dynamo-SGLang AgentX。
@edwingao28 edwingao28 removed the full-sweep-fail-fast Full sweep with canary gate; first failure cancels the rest of that matrix (recommended) label Nov 3, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant