Skip to content

[Docs][History Server] Replace MinIO with RustFS in docs - #5384

Open
fweilun wants to merge 4 commits into
ray-project:masterfrom
fweilun:historyserver-rustfs-docs
Open

fweilun wants to merge 4 commits into
ray-project:masterfrom
fweilun:historyserver-rustfs-docs

Conversation

@fweilun

@fweilun fweilun commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Why are these changes needed?

#5357 switched the History Server e2e tests from MinIO to RustFS and added historyserver/config/rustfs.yaml. The docs still told users to deploy config/minio.yaml, which no longer matched the sample configs.

This PR:

  • Updates README.md, DEVELOPMENT.md, docs/README.md, docs/set_up_collector.md, and docs/set_up_historyserver.md to use RustFS.
  • Removes the unused historyserver/config/minio.yaml.
  • Replaces the MinIO console screenshots with RustFS ones and removes the unused write_logs.png.

Related issue number

Closes #5357.

Checks

  • I've made sure the tests are passing.
  • Testing Strategy
    • Unit tests
    • Manual tests
    • This PR is not tested :(

Manual test instructions

Follow historyserver/docs/set_up_collector.md on a Kind cluster.

grep -rni minio historyserver --exclude=go.sum  # returns empty

fweilun and others added 2 commits October 6, 2026 15:29
@fweilun

fweilun commented Oct 6, 2026

Copy link
Copy Markdown
Contributor Author

@win5923 Please review this PR, thanks.

@win5923

win5923 commented Oct 7, 2026

Copy link
Copy Markdown
Member

I've updated "Follow-up to #5357" to "Closes #5358" so the issue will be closed automatically when the PR is merged.

Comment thread historyserver/docs/set_up_collector.md Outdated
Comment thread historyserver/README.md Outdated
fweilun and others added 2 commits October 6, 2026 23:39
Co-authored-by: Jun-Hao Wan <ken0979172817@gmail.com>
Signed-off-by: fweilun <william20020503@gmail.com>
Co-authored-by: Jun-Hao Wan <ken0979172817@gmail.com>
Signed-off-by: fweilun <william20020503@gmail.com>
@fweilun

fweilun commented Oct 7, 2026

Copy link
Copy Markdown
Contributor Author

Thanks for reviewing this PR!

@win5923 win5923 added the lgtm Ready for final review / merge label Oct 7, 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

lgtm Ready for final review / merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants