From fda93adafeb49ec7229a5daf9e1ccb57e57e235b Mon Sep 17 00:00:00 2001 From: Zhengchao An Date: Wed, 26 Aug 2026 22:26:02 +0800 Subject: [PATCH] fix(ecstore): migrate test-only client import (#6678) --- crates/ecstore/src/store/init.rs | 3 ++- scripts/check_architecture_migration_rules.sh | 6 ++++++ 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/crates/ecstore/src/store/init.rs b/crates/ecstore/src/store/init.rs index 21a9bb454..257dd7bcb 100644 --- a/crates/ecstore/src/store/init.rs +++ b/crates/ecstore/src/store/init.rs @@ -700,7 +700,6 @@ mod tests { validate_durable_ilm_record, }, bucket::metadata::{BUCKET_LIFECYCLE_CONFIG, BUCKET_VERSIONING_CONFIG}, - client::transition_api::ReaderImpl, config::com, core::pools::DecomBucketInfo, data_movement::SourceCleanupDeleteBarrier, @@ -750,6 +749,8 @@ mod tests { #[cfg(feature = "test-util")] use rustfs_protos::{TIER_MUTATION_RPC_PROTOCOL_VERSION, TierMutationRpcPhase}; use rustfs_rio::{Checksum, ChecksumType}; + #[cfg(feature = "test-util")] + use rustfs_s3_client::transition_api::ReaderImpl; use rustfs_utils::{ CompressionAlgorithm, http::{SUFFIX_COMPRESSION, insert_str}, diff --git a/scripts/check_architecture_migration_rules.sh b/scripts/check_architecture_migration_rules.sh index d36cf982f..b7af64cf7 100755 --- a/scripts/check_architecture_migration_rules.sh +++ b/scripts/check_architecture_migration_rules.sh @@ -641,6 +641,12 @@ for ecstore_private_module in \ "mod ${ecstore_private_module};" \ "ECStore legacy ${ecstore_private_module} root module crate-private visibility" done +ECSTORE_CLIENT_SHIM_IMPORT_HITS_FILE="${TMP_DIR}/ecstore_client_shim_import_hits.txt" +if rg -n --no-heading --glob '*.rs' 'crate::client::|^[[:space:]]+client::' \ + "${ROOT_DIR}/crates/ecstore/src" >"${ECSTORE_CLIENT_SHIM_IMPORT_HITS_FILE}"; then + report_failure "ECStore client shim imports must use rustfs_s3_client directly" + cat "${ECSTORE_CLIENT_SHIM_IMPORT_HITS_FILE}" >&2 +fi require_source_line \ "crates/storage-api/src/lib.rs" \ "pub use bucket::{BucketInfo, BucketOperations, BucketOptions, DeleteBucketOptions, MakeBucketOptions, SRBucketDeleteOp};" \