Skip to content

fix(reconcile): treat path-only drive endpoints as one local server - #294

Open
shurkus wants to merge 1 commit into
rustfs:mainfrom
shurkus:fix/issue-293-pdb-single-node-endpoints
Open

shurkus wants to merge 1 commit into
rustfs:mainfrom
shurkus:fix/issue-293-pdb-single-node-endpoints

Conversation

@shurkus

@shurkus shurkus commented Nov 4, 2026 •

Copy link
Copy Markdown
Contributor

Type of Change

  • New Feature
  • Bug Fix
  • Documentation
  • Performance Improvement
  • Test/CI
  • Refactor
  • Other:

Related Issues

Closes #293

Summary of Changes

RustFS reports the drives of a single-node deployment by path only (/data/rustfs0). The PodDisruptionBudget step rejected such a layout ("drive endpoint has no host"), logged a warning and requeued every 60 seconds. Path-only drive endpoints now map to one local server, so a single-server pool resolves to "no budget" without a warning or requeue.

Checklist

  • I have read and followed the CONTRIBUTING.md guidelines
  • Passed make pre-commit (fmt-check + clippy + test + console-lint + console-fmt-check)
  • Added/updated necessary tests
  • Documentation updated (if needed)
  • CHANGELOG.md updated under [Unreleased] (if user-visible change)
  • CI/CD passed (if applicable)

Impact

  • Breaking change (CRD/API compatibility)
  • Requires doc/config/deployment update
  • Other impact: N/A

Verification

make pre-commit
cargo test --lib pod_disruption_budget

Additional Notes

  • Documentation already states that single-server pools get no budget; no change needed.
  • No CHANGELOG entry: the PodDisruptionBudget feature is still under [Unreleased] and this fixes it before release.
  • Found on three single-server Tenants running main 298f568; multi-server Tenants report http://host:port/path endpoints and are unaffected.
  • single_node_pool_with_path_endpoints_gets_no_budget fails on the previous code (60 s requeue) and passes now.

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.

PodDisruptionBudget step warns and requeues forever on single-server Tenants

1 participant