Skip to content

s3: test conditional put_object() and delete_object()#673

Merged
AliMasarweh merged 9 commits intomasterfrom
wip-s3-conditional-write
Jul 28, 2025
Merged

s3: test conditional put_object() and delete_object()#673
AliMasarweh merged 9 commits intomasterfrom
wip-s3-conditional-write

Conversation

@AliMasarweh
Copy link
Member

test coverage for ceph/ceph#63348

  • put_object() with IfMatch and IfNoneMatch
  • delete_object() with IfMatch
  • delete_object() with IfMatchLastModifiedTime and IfMatchSize
  • delete_objects() with <ETag>
  • delete_objects() with <LastModifiedTime>
  • delete_objects() with <Size>

put_object tests are duplicated to test non-versioned and overwrites of a current version
delete_object(s) tests are duplicated to test non-versioned, deletes of a current version, and deletes of a specific version

@AliMasarweh AliMasarweh force-pushed the wip-s3-conditional-write branch 20 times, most recently from f74d07c to 7b7e4a5 Compare July 21, 2025 09:21
cbodley added 7 commits July 21, 2025 12:35
Signed-off-by: Casey Bodley <cbodley@redhat.com>
Signed-off-by: Casey Bodley <cbodley@redhat.com>
Signed-off-by: Casey Bodley <cbodley@redhat.com>
Signed-off-by: Casey Bodley <cbodley@redhat.com>
Signed-off-by: Casey Bodley <cbodley@redhat.com>
Signed-off-by: Casey Bodley <cbodley@redhat.com>
Signed-off-by: Casey Bodley <cbodley@redhat.com>
@AliMasarweh AliMasarweh force-pushed the wip-s3-conditional-write branch 2 times, most recently from 7ed5d02 to 6ba7bd5 Compare July 21, 2025 14:40
we expected an error if we delete non-existent objects but that is not
true

Signed-off-by: Ali Masarwa <ali.saed.masarwa@gmail.com>
@AliMasarweh AliMasarweh force-pushed the wip-s3-conditional-write branch 7 times, most recently from 93afdd1 to b4e9ebf Compare July 27, 2025 13:32
Signed-off-by: Ali Masarwa <ali.saed.masarwa@gmail.com>
@AliMasarweh AliMasarweh force-pushed the wip-s3-conditional-write branch from b4e9ebf to 8be24ac Compare July 27, 2025 13:33
@AliMasarweh AliMasarweh merged commit 538dedc into master Jul 28, 2025
@ivancich
Copy link
Member

@cbodley : What's our policy for s3-tests PRs. Is a review with approval required? I looked at some other recently merged PRs and they all had a review with an approval.

@cbodley
Copy link
Contributor

cbodley commented Jul 31, 2025

@cbodley : What's our policy for s3-tests PRs. Is a review with approval required? I looked at some other recently merged PRs and they all had a review with an approval.

no real policy. i often have to merge my own test prs without review

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.

3 participants