Project

General

Profile

Bug #58287

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

Added by Matt Benjamin 8 months ago. Updated 3 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

Duplicated by rgw - Bug #58282: zero size is reported for lifecycle notifications on versioned buckets Duplicate
Copied to rgw - Backport #59012: quincy: rgwlc/notifications: initialize object state before accessing object size for use in notify New
Copied to rgw - Backport #59013: reef: rgwlc/notifications: initialize object state before accessing object size for use in notify Resolved
Copied to rgw - Backport #61370: pacific: rgwlc/notifications: initialize object state before accessing object size for use in notify New

History

#1 Updated by Matt Benjamin 8 months ago

  • Pull request ID set to 49466

#2 Updated by Matt Benjamin 8 months ago

  • Status changed from In Progress to Fix Under Review
  • Backport set to quincy

#3 Updated by Casey Bodley 7 months ago

  • Duplicated by Bug #58282: zero size is reported for lifecycle notifications on versioned buckets added

#4 Updated by Casey Bodley 5 months ago

  • Status changed from Fix Under Review to Pending Backport
  • Tags set to lifecycle notification
  • Backport changed from quincy to quincy reef

#5 Updated by Backport Bot 5 months ago

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

#6 Updated by Backport Bot 5 months ago

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

#7 Updated by Backport Bot 5 months ago

  • Tags changed from lifecycle notification to lifecycle notification backport_processed

#8 Updated by Casey Bodley 3 months ago

  • Tags changed from lifecycle notification backport_processed to lifecycle notification
  • Backport changed from quincy reef to pacific quincy reef

#9 Updated by Backport Bot 3 months ago

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

#10 Updated by Backport Bot 3 months ago

  • Tags changed from lifecycle notification to lifecycle notification backport_processed

Also available in: Atom PDF