From b821684caae1fdf287610eb65effe310b0af198a Mon Sep 17 00:00:00 2001 From: cxymds Date: Thu, 5 Nov 2026 09:14:13 +0800 Subject: [PATCH] test(ci): refresh e2e selections for duplicate CreateBucket The two new create_bucket_region cases join the e2e-smoke and e2e-full selections, which main also extended with multipart_upload_id tests. smoke is measured from a local nextest listing: removing the two new cases reproduces main's committed digest exactly. e2e-full is derived the same way from main's listing artifact (run 41933823399, babfeaed0) plus the two multipart_upload_id cases, which reproduces main's committed linux digest before the new cases are added; CI re-verifies both platforms. test_bucket_create_exists_nonowner asserts BucketAlreadyExists for a second identity, which a single-owner deployment cannot produce. Move it to excluded_tests.txt with the deployment-ownership rationale; the IAM-user duplicate paths stay covered by crates/e2e_test/src/create_bucket_region_test.rs. --- .config/e2e-full-selection.txt | 4 ++-- .config/e2e-smoke-selection.txt | 2 +- scripts/s3-tests/excluded_tests.txt | 8 ++++++++ scripts/s3-tests/implemented_tests.txt | 1 - 4 files changed, 11 insertions(+), 4 deletions(-) diff --git a/.config/e2e-full-selection.txt b/.config/e2e-full-selection.txt index a3d621f42..a2a1431f1 100644 --- a/.config/e2e-full-selection.txt +++ b/.config/e2e-full-selection.txt @@ -1,2 +1,2 @@ -sha256-darwin=7c705942d3a9e583ebaa85845daa21c4f81fdbf36def24db931aaef38636c53e -sha256-linux=01e1f5569be60674e30d9e30a9961c5c53a6cea6ca1f7913218f2adf7291d574 +sha256-darwin=a3e166a140f3150218b4a857fedb8749940fad0c8b053d0e3c1b957321bf5beb +sha256-linux=dce4a8eb634a27b4589bcf6d2ebf248ede97aa59701acc8696391d2812a3177d diff --git a/.config/e2e-smoke-selection.txt b/.config/e2e-smoke-selection.txt index aef666429..1453835a5 100644 --- a/.config/e2e-smoke-selection.txt +++ b/.config/e2e-smoke-selection.txt @@ -1 +1 @@ -sha256=f794c004e5952147c79dadbe6b7c84804bc986f34a8142731fac8864004e5130 +sha256=5bdd4c2eadbc41625ae3184fff34062e000d328108436259b2b7c4fe2531ecaf diff --git a/scripts/s3-tests/excluded_tests.txt b/scripts/s3-tests/excluded_tests.txt index 1e2e19180..0fb390bc9 100644 --- a/scripts/s3-tests/excluded_tests.txt +++ b/scripts/s3-tests/excluded_tests.txt @@ -315,3 +315,11 @@ test_lifecycle_cloud_transition_target_by_bucket test_lifecycle_cloud_transition_target_by_bucket_multiple_buckets test_list_object_versions_restore_status test_list_objects_restore_status + +# Require per-account bucket ownership across distinct S3 accounts +# A RustFS deployment owns one bucket namespace and records no bucket owner, so every +# authorized requester is the bucket's owner: a duplicate CreateBucket returns the +# legacy 200 in the default region and BucketAlreadyOwnedByYou elsewhere, never +# BucketAlreadyExists for a second account. The deployment-level behavior is covered by +# crates/e2e_test/src/create_bucket_region_test.rs. +test_bucket_create_exists_nonowner diff --git a/scripts/s3-tests/implemented_tests.txt b/scripts/s3-tests/implemented_tests.txt index cfe9df930..d24542c2c 100644 --- a/scripts/s3-tests/implemented_tests.txt +++ b/scripts/s3-tests/implemented_tests.txt @@ -205,7 +205,6 @@ test_put_public_block test_block_public_policy test_block_public_policy_with_principal test_block_public_restrict_public_buckets -test_bucket_create_exists_nonowner test_get_public_block_deny_bucket_policy test_get_undefined_public_block