Skip to content

Retire the MI300X TW runner pool - #3436

Merged
cquil11 merged 1 commit into
mainfrom
codex/retire-mi300x-tw
Sep 25, 2026
Merged

cquil11 merged 1 commit into
mainfrom
codex/retire-mi300x-tw

Conversation

@cquil11

@cquil11 cquil11 commented Sep 25, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Remove the retired pool from runner inventory and hardware metadata.
  • Remove its CollectiveX platform and launcher branches while preserving MI325X TW behavior.
  • Update the existing localized documentation.

Validation

  • CollectiveX matrix tests: 8 passed, 100 subtests passed.
  • Bash syntax checks and git diff --check passed.
  • No remaining tracked references to the retired pool.

AI model disclosure

Prepared by OpenAI Codex. The exact model version could not be verified from the runtime.

@cquil11
cquil11 requested a review from a team September 25, 2026 17:06
@cquil11
cquil11 merged commit fb84998 into main Sep 25, 2026
2 checks passed
@cquil11
cquil11 deleted the codex/retire-mi300x-tw branch September 25, 2026 17:06

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

Looks good, straightforward retirement/cleanup change. Reviewed the runner/hardware label removal in configs/runners.yaml, the platform block removal in platform_config.json, the case-arm removal and SDMA workaround deletion in launch_mi-tw.sh, the case-pattern trim in launch_swap-blocks.sh, and the doc wording updates (both languages) — confirmed no remaining references to mi300x-tw anywhere in the repo.

Extended reasoning...

Six-file diff retiring the mi300x-tw AMD runner pool: removes it from runner/hardware config, CollectiveX platform config, two launcher scripts' case statements (including a now-obsolete SDMA workaround), and updates English/Chinese docs from plural to singular pool wording. No security-sensitive surface (auth, crypto, permissions) is touched; it's pure deletion of dead branches/config with no new logic. A repo-wide grep confirms no leftover mi300x-tw references, matching the PR's own validation claim, and no bug-hunter findings or outstanding reviewer objections exist in the timeline.

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

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant