Skip to content

test(admin): cover real backend ILM job commands - #337

Merged
houseme merged 1 commit into
mainfrom
houseme/ilm-transition-real-backend-e2e
Jul 26, 2026
Merged

test(admin): cover real backend ILM job commands#337
houseme merged 1 commit into
mainfrom
houseme/ilm-transition-real-backend-e2e

Conversation

@houseme

@houseme houseme commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add an ignored real-backend integration test for rc admin ilm transition run --async, status, wait, and cancel.
  • Start a local RustFS child process from RC_REAL_RUSTFS_BINARY, configure RC_HOST_realilm, and verify the CLI talks to the real admin backend instead of the fake test server.
  • Assert that terminal cancel does not rewrite a finished job into cancelled.

Verification

  • cargo fmt --all --check
  • cargo clippy -p rustfs-cli --test admin_ilm_real_backend -- -D warnings
  • cargo clippy --workspace -- -D warnings
  • cargo test -p rustfs-cli --test admin_ilm_real_backend -- --list
  • RC_REAL_RUSTFS_BINARY=/private/tmp/rustfs-1479-task-journal-attribution/target/debug/rustfs cargo test -p rustfs-cli --test admin_ilm_real_backend -- --ignored --nocapture
  • cargo test --workspace

Notes

  • The real-backend test is ignored by default because it requires an external RustFS binary via RC_REAL_RUSTFS_BINARY.

Co-Authored-By: heihutu <heihutu@gmail.com>
@houseme
houseme merged commit ef82a1d into main Jul 26, 2026
17 checks passed
@houseme
houseme deleted the houseme/ilm-transition-real-backend-e2e branch July 26, 2026 12:54
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