Skip to content

feat(rc-admin)!: add scanner, storage, and metrics diagnostics#277

Closed
cxymds wants to merge 2 commits into
cxymds/issue-1362-output-v3from
cxymds/issue-1376-scanner-metrics
Closed

feat(rc-admin)!: add scanner, storage, and metrics diagnostics#277
cxymds wants to merge 2 commits into
cxymds/issue-1362-output-v3from
cxymds/issue-1376-scanner-metrics

Conversation

@cxymds

@cxymds cxymds commented Jul 21, 2026

Copy link
Copy Markdown
Member

Related issue

Closes rustfs/backlog#1376

Background and impact

RustFS beta.10 exposes scanner status, storage information, and realtime metrics through Admin API v3, but rc had no typed client boundary or safe CLI surface for these diagnostics. Operators could not distinguish stale, empty, partial, unauthorized, and unsupported states in automation.

Solution

  • add typed core contracts for scanner, storage, and realtime metric snapshots
  • add signed Admin API clients for /scanner/status, /storageinfo, and streaming /metrics
  • bound metrics to 120 snapshots, 1 MiB per record, and 16 MiB per response
  • add rc admin scanner status, rc admin info storage, and rc admin metrics
  • normalize metrics into v3 JSON Lines while preserving raw server snapshots, numeric values, labels, and timestamps
  • distinguish permission failures from missing server routes
  • extend output v3 schema, golden fixtures, help contracts, and reference documentation

BREAKING

This PR updates the protected rc admin behavior reference and extends the output v3 metrics contract with errors, partial/final markers, raw data, and per-sample timestamps. Consumers of the previously provisional metrics v3 shape must adopt the documented fields. Output v1 and v2 remain unchanged.

Validation

  • cargo fmt --all --check
  • cargo clippy --workspace --all-targets -- -D warnings
  • cargo test --workspace
  • cargo test -p rustfs-cli --features integration --test integration --no-run

All validation commands pass locally.

@cxymds
cxymds changed the base branch from cxymds/issue-1363-capabilities to main July 21, 2026 11:24
@cxymds cxymds closed this Jul 21, 2026
@cxymds cxymds reopened this Jul 21, 2026
@cxymds
cxymds changed the base branch from main to cxymds/issue-1363-capabilities July 21, 2026 11:24
@cxymds

cxymds commented Jul 21, 2026

Copy link
Copy Markdown
Member Author

Exact-head validation is complete for 8ec668b483c600786df8d25871cd3387e2e36c3b.

The PR remains draft because it is part of the ordered capability/schema stack.

@cxymds
cxymds deleted the branch cxymds/issue-1362-output-v3 July 22, 2026 13:01
Base automatically changed from cxymds/issue-1363-capabilities to cxymds/issue-1362-output-v3 July 22, 2026 13:01
@cxymds cxymds closed this Jul 22, 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