Project

General

Profile

Bug #51520

notifications: send only CompleteMultipartUpload notification

Added by Yuval Lifshitz over 2 years ago. Updated over 1 year ago.

Status:
Pending Backport
Priority:
Normal
Target version:
-
% Done:

0%

Source:
Q/A
Tags:
notifications backport_processed
Backport:
pacific
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

existing behavior
  • first notifications is: s3:ObjectCreated:Post
  • every part uploaded will send: s3:ObjectCreated:Put
  • when upload is complete, you will get: s3:ObjectCreated:CompleteMultipartUpload

requirements
  • the POST notification sent at the beginning of the multipart upload should not be sent
  • the intermediate PUT notifications sent during the multipart upload should not be sent either

testing

code

documentation:
add a note here: https://github.com/ceph/ceph/blob/master/doc/radosgw/s3-notification-compatibility.rst#event-types
explaining the the behavior in case of multipart upload.

release notes
since this is a change in product behavior, there should be a note added to: https://github.com/ceph/ceph/blob/master/PendingReleaseNotes


Related issues

Copied to rgw - Backport #51943: pacific: notifications: send only CompleteMultipartUpload notification Resolved

History

#1 Updated by Yuval Lifshitz over 2 years ago

existing behavior

  • first notifications is: s3:ObjectCreated:Post
  • every part uploaded will send: s3:ObjectCreated:Put
  • when upload is complete, you will get: s3:ObjectCreated:CompleteMultipartUpload

requirements

  • the POST notification sent at the beginning of the multipart upload should not be sent
  • the intermediate PUT notifications sent during the multipart upload should not be sent either

testing

code

documentation:

add a note here: https://github.com/ceph/ceph/blob/master/doc/radosgw/s3-notification-compatibility.rst#event-types
explaining the the behavior in case of multipart upload.

release notes

since this is a change in product behavior, there should be a note added to: https://github.com/ceph/ceph/blob/master/PendingReleaseNotes

#2 Updated by Yuval Lifshitz over 2 years ago

  • Status changed from New to Fix Under Review
  • Pull request ID set to 42350

#3 Updated by Yuval Lifshitz over 2 years ago

  • Status changed from Fix Under Review to Pending Backport

#4 Updated by Backport Bot over 2 years ago

  • Copied to Backport #51943: pacific: notifications: send only CompleteMultipartUpload notification added

#5 Updated by Backport Bot over 1 year ago

  • Tags changed from notifications to notifications backport_processed

Also available in: Atom PDF