Skip to content

feat(admin): add decommission clear command#240

Merged
overtrue merged 1 commit into
mainfrom
cxymds/admin-decommission-clear
Jun 23, 2026
Merged

feat(admin): add decommission clear command#240
overtrue merged 1 commit into
mainfrom
cxymds/admin-decommission-clear

Conversation

@cxymds

@cxymds cxymds commented Jun 23, 2026

Copy link
Copy Markdown
Member

Related Issue

None.

Background

rc admin decommission supported start, status, and cancel, but it did not expose RustFS' pool decommission clear operation. Operators need this command to clear failed or canceled decommission metadata by pool, including by pool ID.

Root Cause

The admin API trait and S3 admin client did not include a decommission_clear operation, and the CLI decommission subcommands did not include clear.

Solution

  • Add rc admin decommission clear <ALIAS> <POOL> [--by-id].
  • Dispatch clear requests to POST /rustfs/admin/v3/pools/clear with the existing pool query shape.
  • Add parser, help contract, CLI dispatch, and S3 route coverage.
  • Extend the shared admin test server accept deadline to avoid false failures when spawned CLI tests start slowly under local load.

Test Status

  • cargo fmt --all --check
  • cargo clippy --workspace -- -D warnings
  • cargo test --workspace

@cxymds cxymds requested a review from overtrue June 23, 2026 07:07
@overtrue overtrue merged commit ab0cded into main Jun 23, 2026
15 checks passed
@overtrue overtrue deleted the cxymds/admin-decommission-clear branch June 23, 2026 08:49
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