Skip to content

Add redirects for removed gpu-arch pages to ROCm docs - #134

Merged
AMD-melliott merged 1 commit into
ROCm:developfrom
AMD-melliott:gpu-arch-redirects
Aug 27, 2026
Merged

Add redirects for removed gpu-arch pages to ROCm docs#134
AMD-melliott merged 1 commit into
ROCm:developfrom
AMD-melliott:gpu-arch-redirects

Conversation

@AMD-melliott

@AMD-melliott AMD-melliott commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • The GPU architecture pages (gpu-arch/gpu-arch, mi100, mi250, mi300, mi300-mi200-performance-counters, mi350) were removed in Remove gpu-arch pages, link to ROCm docs instead #132 in favor of linking out to rocm.docs.amd.com, but the old instinct-docs URLs now 404 instead of forwarding readers to the live pages.
  • Wires up sphinx-reredirects (already a pinned but unused dependency in docs/sphinx/requirements.txt) with a mapping from each removed docname to its ROCm-hosted equivalent.

Test plan

  • Verified redirect generation mechanics with an isolated minimal Sphinx build using the same docnames and redirects config — all 6 files generated at the expected build paths with correct meta-refresh/JS targets.
  • Confirm on the deployed preview/RTD build that /gpu-arch/<page>.html forwards to the corresponding rocm.docs.amd.com/en/latest/reference/gpu-arch/... page.

@AMD-melliott

Copy link
Copy Markdown
Collaborator Author

The GPU architecture pages were deleted in favor of linking out to
rocm.docs.amd.com (ROCm#132), but the old instinct-docs URLs still 404
instead of forwarding readers to the live equivalents. Wire up the
already-pinned sphinx-reredirects extension with a mapping from each
removed docname to its ROCm-hosted replacement.
@AMD-melliott
AMD-melliott merged commit 5c4cc50 into ROCm:develop Aug 27, 2026
3 checks passed
@AMD-melliott
AMD-melliott deleted the gpu-arch-redirects branch August 27, 2026 20:23
AMD-melliott added a commit to AMD-melliott/instinct-docs that referenced this pull request Aug 28, 2026
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.

1 participant