Project

General

Profile

Actions

Bug #63532

open

notification: etag is missing in CompleteMultipartUpload event

Added by Yuval Lifshitz 6 months ago. Updated about 1 month ago.

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

0%

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

Description

Body:
{"Records":[{"eventVersion":"2.2","eventSource":"ceph:s3","awsRegion":"default","eventTime":"2023-11-14T16:32:56.594206Z","eventName":"ObjectCreated:CompleteMultipartUpload","userIdentity":{"principalId":"testid"},"requestParameters":{"sourceIPAddress":""},"responseElements":{"x-amz-request-id":"dcfcc511-002b-4b75-a6b3-63fd942a6610.4147.8543484732195856827","x-amz-id-2":"1033-default-default"},"s3":{"s3SchemaVersion":"1.0","configurationId":"notif1","bucket":{"name":"fish","ownerIdentity":{"principalId":"testid"},"arn":"arn:aws:s3:default::fish","id":"dcfcc511-002b-4b75-a6b3-63fd942a6610.4149.1"},"object":{"key":"myfile","size":104857600,"eTag":"","versionId":"","sequencer":"38A15365A9E53B24","metadata":[],"tags":[]}},"eventId":"1699979576.607905.","opaqueData":""}]}

Related issues 2 (1 open1 closed)

Copied to rgw - Backport #63856: reef: notification: etag is missing in CompleteMultipartUpload eventResolvedYuval LifshitzActions
Copied to rgw - Backport #63857: quincy: notification: etag is missing in CompleteMultipartUpload eventNewAli MasarwaActions
Actions #1

Updated by Yuval Lifshitz 6 months ago

  • Status changed from New to Fix Under Review
  • Backport changed from reef, pacific to reef, quincy
  • Pull request ID set to 54528
Actions #2

Updated by Yuval Lifshitz 6 months ago

  • Status changed from Fix Under Review to New
  • Pull request ID deleted (54528)
Actions #3

Updated by Yuval Lifshitz 6 months ago

  • Assignee changed from Yuval Lifshitz to Ali Masarwa

the "publish_commit()" should be moved from "execute()" to "commit()" where the final etag is calculated.

(based on the fix from here: https://github.com/ceph/ceph/pull/50627 the etag has value only in "complete()")

Actions #4

Updated by Yuval Lifshitz 5 months ago

  • Status changed from New to Pending Backport
Actions #5

Updated by Backport Bot 5 months ago

  • Copied to Backport #63856: reef: notification: etag is missing in CompleteMultipartUpload event added
Actions #6

Updated by Backport Bot 5 months ago

  • Copied to Backport #63857: quincy: notification: etag is missing in CompleteMultipartUpload event added
Actions #7

Updated by Backport Bot 5 months ago

  • Tags changed from notifications to notifications backport_processed
Actions #8

Updated by Yuval Lifshitz about 1 month ago

  • Pull request ID set to 54569
Actions

Also available in: Atom PDF