Skip to content

docs: P0 — fix factual errors, dangerous ops guidance, leftover test content - #99

Merged
overtrue merged 1 commit into
mainfrom
overtrue/docs-p0-accuracy-fixes
Jul 16, 2026
Merged

docs: P0 — fix factual errors, dangerous ops guidance, leftover test content#99
overtrue merged 1 commit into
mainfrom
overtrue/docs-p0-accuracy-fixes

Conversation

@overtrue

@overtrue overtrue commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator

Summary (review workstream: P0 accuracy & safety — rustfs/backlog#1277)

Fixes the highest-risk factual errors and dangerous operational guidance found in the docs review (25 files + 1 test directory removed).

Dangerous guidance (following it caused incidents):

  • Disk replacement: mkfs.ext4mkfs.xfs; mount points aligned with the install layout (healing never triggered as written)
  • Pool expansion: unified RUSTFS_VOLUMES hostname patterns across old/new nodes (as written, the whole cluster failed to start); "rolling restart prohibited" narrowed to "only pool changes need a full restart"; systemd TimeoutStartSec 30s → 120s (matches the upstream unit; 30s causes kill-restart loops on cold start)
  • Nginx: added TLS directives to the 443 example; moved the S3 API off the /api path prefix (SigV4 breaks under a prefix); replaced the conflicting Connection header pairs with an $http_upgrade map so upstream keepalives survive while websocket upgrades still work — incorporates Fix nginx websocket and keepalive support #76 by @jstangroome (the original PR targets the pre-migration docs/ path)
  • Firewall sections now open 9001 (Console) alongside 9000

Fabricated content removed: rustfs-admin / rc command families (self-described as "hypothetical"), nonexistent config.toml/config.yaml, nonexistent Operator/LogSearch/Elasticsearch/NSQ (real Pulsar target added); developer/mcp.md now warns that crates/mcp moved out of the main repo.

Hard numbers corrected: single PUT limit 500 GiB → 5 GiB (the old number was MinIO's); EC read/write quorum formulas (read = N−parity, write = data shards); Blake3 → HighwayHash256; console port 9090 → 9001; desktop launcher port 7001 documented as distinct from the server default 9001; io_uring claims aligned with source (opt-in, off by default).

Hygiene: hand-signed SigV4 curl examples → aws s3api; private paths / password.txt / realistic-looking access keys / SSH clone URLs removed; leftover test page content/deep/ deleted; broken #mode anchors fixed; "100% secure / 100% S3 compatible" claims removed.

Every correction is backed by a source citation (file:line) in the review reports (see the nav PR's docs-review/). Build: ✓ 355 pages.


Backed by the multi-role docs review (master issue rustfs/backlog#1277; six full reports ship in the nav PR under docs-review/). Technical facts verified against rustfs/rustfs@ea2e24ac.

Rendered page previews (before / after)

/concepts/limit
Before After
before after
/concepts/principle/erasure-coding
Before After
before after
/developer/mcp
Before After
before after
/installation/checklists/network-checklists
Before After
before after
/installation/checklists/security-checklists
Before After
before after
/installation/checklists/software-checklists
Before After
before after
/installation/linux/multiple-node-multiple-disk
Before After
before after
/installation/linux/single-node-multiple-disk
Before After
before after
/installation/linux/single-node-single-disk
Before After
before after
/integration/nginx
Before After
before after
/integration/tls-configured
Before After
before after
/integration/virtual
Before After
before after
/management/bucket/creation
Before After
before after
/management/bucket/deletion
Before After
before after
/management/object/creation
Before After
before after
/management/object/deletion
Before After
before after
/troubleshooting/driver
Before After
before after
/troubleshooting/healing
Before After
before after
/troubleshooting/node
Before After
before after
/upgrade-scale/availability-and-resiliency
Before After
before after
/features/encryption (new page)

new

/features/logging (new page)

new

/installation/linux (new page)

new

/installation/macos (new page)

new

/installation/windows (new page)

new

…ontent

- Remove fabricated CLI commands (rustfs-admin, rc) and nonexistent config files (config.toml/config.yaml)
- Fix disk-replacement guide: mkfs.ext4 -> mkfs.xfs, align mount points with install layout
- Fix expansion guide: unify RUSTFS_VOLUMES hostname patterns, correct rolling-restart semantics, systemd TimeoutStartSec 30s -> 120s (matches upstream unit)
- Correct console port 9090 -> 9001; open 9001 in firewall sections; document desktop launcher port 7001 vs server default 9001
- concepts/limit: single PUT 500 GiB -> 5 GiB; fix EC read/write quorum formulas
- erasure-coding: Blake3 -> HighwayHash256 (matches bitrot implementation)
- logging: resolve Prometheus self-contradiction; remove nonexistent Operator/LogSearch/Elasticsearch/NSQ, add Pulsar
- nginx: add TLS directives to 443 example; move S3 off path prefix (SigV4 breaks under /api)
- Replace hand-signed SigV4 curl examples with aws s3api; drop private paths and password.txt samples
- mcp: warn that crates/mcp moved out of the main repo; HTTPS clone URLs
- Remove leftover test page content/deep; drop io_uring claims that contradict upstream; remove Serf/VxLAN port requirements; fix broken #mode anchors; remove 100%-secure/100%-compatible claims
Copilot AI review requested due to automatic review settings July 15, 2026 07:02
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
docs-rustfs-com 836a2b0 Commit Preview URL

Branch Preview URL
Jul 15 2026, 07:01 AM

@vercel

vercel Bot commented Jul 15, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs-rustfs-com Ready Ready Preview, Comment Jul 15, 2026 7:02am

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@overtrue
overtrue merged commit 8bfd73f into main Jul 16, 2026
3 checks passed
@overtrue
overtrue deleted the overtrue/docs-p0-accuracy-fixes branch July 16, 2026 03:56
overtrue pushed a commit to jstangroome/docs.rustfs.com that referenced this pull request Jul 16, 2026
By default nginx sends a Connection: close request header upstream,
which breaks upstream keepalives (latency from repeated TCP handshakes,
port exhaustion under load). An http_upgrade map sends a blank
Connection header to preserve keepalives, or Connection: upgrade when
the client requests a websocket upgrade.

Rebased onto content/integration/nginx.md after the VitePress to
FumaPress migration and the rustfs#99 restructure.
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.

2 participants