Project

General

Profile

Actions

Bug #54528

open

cls_rgw_suggest_changes() may accidentally delete newly-completed entries

Added by Casey Bodley about 2 years ago. Updated over 1 year ago.

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

0%

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

Description

if bucket listing races with a pending index transaction, its suggested removal may be mistakenly applied if that index transaction completes before the osd receives this suggestion

in rgw_dir_suggest_changes(), the sole condition for applying a suggested change is that the cur_disk.pending_map is empty. this is true after rgw_bucket_complete_op()

racing sequence of events in cls/rgw:

rgw_bucket_prepare_op() adds entry to pending_map

rgw_bucket_list() returns entry with pending_map

rgw_bucket_complete_op() completes entry, clears pending_map

rgw_dir_suggest_changes() finds pending_map empty, so applies a suggestion to remove the listed entry


Related issues 3 (0 open3 closed)

Copied to rgw - Backport #55043: quincy: cls_rgw_suggest_changes() may accidentally delete newly-completed entriesResolvedJ. Eric IvancichActions
Copied to rgw - Backport #55044: pacific: cls_rgw_suggest_changes() may accidentally delete newly-completed entriesResolvedJ. Eric IvancichActions
Copied to rgw - Backport #55045: octopus: cls_rgw_suggest_changes() may accidentally delete newly-completed entriesResolvedJ. Eric IvancichActions
Actions #1

Updated by Casey Bodley about 2 years ago

  • Status changed from Fix Under Review to Pending Backport
Actions #2

Updated by Backport Bot about 2 years ago

  • Copied to Backport #55043: quincy: cls_rgw_suggest_changes() may accidentally delete newly-completed entries added
Actions #3

Updated by Backport Bot about 2 years ago

  • Copied to Backport #55044: pacific: cls_rgw_suggest_changes() may accidentally delete newly-completed entries added
Actions #4

Updated by Backport Bot about 2 years ago

  • Copied to Backport #55045: octopus: cls_rgw_suggest_changes() may accidentally delete newly-completed entries added
Actions #5

Updated by Backport Bot over 1 year ago

  • Tags changed from dir_suggest to dir_suggest backport_processed
Actions

Also available in: Atom PDF