Skip to content

HDDS-15124. [Docs] Fix broken /docs/current/ links in CONTRIBUTING.md and dist/README.md#10153

Merged
adoroszlai merged 1 commit intoapache:masterfrom
chihsuan:HDDS-15124
Apr 29, 2026
Merged

HDDS-15124. [Docs] Fix broken /docs/current/ links in CONTRIBUTING.md and dist/README.md#10153
adoroszlai merged 1 commit intoapache:masterfrom
chihsuan:HDDS-15124

Conversation

@chihsuan
Copy link
Copy Markdown
Contributor

@chihsuan chihsuan commented Apr 28, 2026

What changes were proposed in this pull request?

Updates four broken https://ozone.apache.org/docs/current/...html links that
404 after the docs site moved from Hugo to Docusaurus (the current/ namespace
and .html suffix were dropped).

URL mapping

File:Line Old URL New URL
CONTRIBUTING.md:78 …/docs/current/start.html …/docs/developer-guide/run/docker-compose/
dist/README.md:33 …/docs/current/start/runningviadocker.html …/docs/quick-start/installation/docker/
dist/README.md:41 …/docs/current/start/minikube.html …/docs/quick-start/installation/kubernetes/#minikube
dist/README.md:42 …/docs/current/start/kubernetes.html …/docs/quick-start/installation/kubernetes/

Why these targets

The two files target different audiences, so the same "Docker Compose" topic
maps to different pages:

  • CONTRIBUTING.md:78 — Surrounding text walks contributors through
    building from source and running the in-tree compose. The dev-guide page
    documents that exact flow, opening with the same
    cd ./hadoop-ozone/dist/target/ozone-<VERSION>/compose/ozone command.

  • dist/README.md:33,41,42 — Both the link text ("Getting Started: ...")
    and the surrounding intro ("More information can be found in the Getting Started Guide:") explicitly invoke the Quick Start framing, so the URLs
    match those labels.

Out of scope

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-15124

How was this patch tested?

  • Verified each new URL exists in the apache/ozone-site Docusaurus source:
    • docs/02-quick-start/01-installation/02-kubernetes.md (with a ## Minikube
      heading for the anchor)
    • docs/08-developer-guide/02-run/02-docker-compose.md
    • docs/02-quick-start/01-installation/README.mdx
  • Markdown-only change; no code, config, or test files modified.
  • build-branch CI on the fork: docs / RAT / checkstyle / PMD / findbugs /
    acceptance jobs all green. Two integration jobs hit known flakes:
    • integration (recon) cleared on rerun (transient).
    • integration (hdds)TestDeadNodeHandler.testOnMessage is the failure
      tracked by HDDS-14977 (Open).

… and dist/README.md

Updates four broken https://ozone.apache.org/docs/current/...html documentation
links that returned 404 after the docs site moved from Hugo to Docusaurus
(eliminating the current/ namespace and .html suffix).

Each link is repointed at the equivalent Docusaurus page, choosing targets that
match the audience and link-text framing of the surrounding paragraph:

- CONTRIBUTING.md:78 (Docker Compose) -> /docs/developer-guide/run/docker-compose/
  The surrounding text walks contributors through building from source and
  running the in-tree compose; the dev-guide page documents that exact flow,
  starting with the same `cd ./hadoop-ozone/dist/target/ozone-<VERSION>/compose/ozone`
  step.

- dist/README.md:33,41,42 -> /docs/quick-start/installation/{docker,kubernetes,kubernetes/#minikube}
  Both the link text ("Getting Started: ...") and the surrounding intro
  ("More information can be found in the Getting Started Guide:") explicitly
  invoke the Quick Start framing, so the URLs match those labels.

Out of scope: broken /docs/edge/...html links and the OEP link on
CONTRIBUTING.md:98 -- both are covered by HDDS-14663 / apache/ozone-site#351,
which restores /docs/edge/ as a frozen archive.
Copy link
Copy Markdown
Contributor

@jojochuang jojochuang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. I wanted to restore the current link in the site doc but that PR is not being reviewed.

@adoroszlai adoroszlai merged commit 72e69f0 into apache:master Apr 29, 2026
18 checks passed
@adoroszlai
Copy link
Copy Markdown
Contributor

Thanks @chihsuan for the patch, @jojochuang for the review.

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.

3 participants