Project

General

Profile

Actions
Copy link

Bug #58287

open

rgwlc/notifications: initialize object state before accessing object size for use in notify

Added by Matt Benjamin over 1 year ago. Updated 11 months ago.

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

0%

Source:
Tags:
lifecycle notification backport_processed
Backport:
pacific quincy reef
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
49466
Crash signature (v1):
Crash signature (v2):

Description

In remove_expired_obj(...), the value of obj->get_obj_size() is sent as the value for object size by the corresponding notification, but that is guaranteed to be 0 if obj->get_obj_state(...) has not been previously called (it hasn't).

This results in lc notifies sending 0 uniformly for obj_size, rather than the stored size.

Related issues 4 (1 open — 3 closed)

Has duplicate rgw - Bug #58282: zero size is reported for lifecycle notifications on versioned bucketsDuplicateYuval Lifshitz

Actions
Copied to rgw - Backport #59012: quincy: rgwlc/notifications: initialize object state before accessing object size for use in notifyNewMatt BenjaminActions
Copied to rgw - Backport #59013: reef: rgwlc/notifications: initialize object state before accessing object size for use in notifyResolvedMatt BenjaminActions
Copied to rgw - Backport #61370: pacific: rgwlc/notifications: initialize object state before accessing object size for use in notifyRejectedMatt BenjaminActions
Actions
Copy link
 #1

Updated by Matt Benjamin over 1 year ago

  • Pull request ID set to 49466
Actions
Copy link
 #2

Updated by Matt Benjamin over 1 year ago

  • Status changed from In Progress to Fix Under Review
  • Backport set to quincy
Actions
Copy link
 #3

Updated by Casey Bodley over 1 year ago

  • Has duplicate Bug #58282: zero size is reported for lifecycle notifications on versioned buckets added
Actions
Copy link
 #4

Updated by Casey Bodley about 1 year ago

  • Status changed from Fix Under Review to Pending Backport
  • Tags set to lifecycle notification
  • Backport changed from quincy to quincy reef
Actions
Copy link
 #5

Updated by Backport Bot about 1 year ago

  • Copied to Backport #59012: quincy: rgwlc/notifications: initialize object state before accessing object size for use in notify added
Actions
Copy link
 #6

Updated by Backport Bot about 1 year ago

  • Copied to Backport #59013: reef: rgwlc/notifications: initialize object state before accessing object size for use in notify added
Actions
Copy link
 #7

Updated by Backport Bot about 1 year ago

  • Tags changed from lifecycle notification to lifecycle notification backport_processed
Actions
Copy link
 #8

Updated by Casey Bodley 11 months ago

  • Tags changed from lifecycle notification backport_processed to lifecycle notification
  • Backport changed from quincy reef to pacific quincy reef
Actions
Copy link
 #9

Updated by Backport Bot 11 months ago

  • Copied to Backport #61370: pacific: rgwlc/notifications: initialize object state before accessing object size for use in notify added
Actions
Copy link
 #10

Updated by Backport Bot 11 months ago

  • Tags changed from lifecycle notification to lifecycle notification backport_processed
Actions
Copy link

Also available in: Atom PDF