Project

General

Profile

Actions

Bug #18076

closed

multisite: some yields in RGWMetaSyncShardCR::full_sync() resume in incremental_sync()

Added by Casey Bodley over 7 years ago. Updated almost 7 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Target version:
-
% Done:

0%

Source:
Tags:
Backport:
jewel, kraken
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

When RGWMetaSyncShardCR::full_sync() is done, it updates the sync_marker and writes it to rados. But after it changes 'sync_marker.state = IncrementalSync;', any yields will cause RGWMetaSyncShardCR::operate() to call incremental_sync() instead - so most of the cleanup and error paths in full_sync() are unreachable.


Related issues 2 (0 open2 closed)

Copied to rgw - Backport #19048: jewel: multisite: some yields in RGWMetaSyncShardCR::full_sync() resume in incremental_sync()ResolvedNathan CutlerActions
Copied to rgw - Backport #19049: kraken: multisite: some yields in RGWMetaSyncShardCR::full_sync() resume in incremental_sync()ResolvedNathan CutlerActions
Actions #1

Updated by Casey Bodley over 7 years ago

  • Backport changed from jewel to jewel, kraken
Actions #2

Updated by Casey Bodley about 7 years ago

  • Status changed from New to Pending Backport
Actions #3

Updated by Loïc Dachary about 7 years ago

  • Copied to Backport #19048: jewel: multisite: some yields in RGWMetaSyncShardCR::full_sync() resume in incremental_sync() added
Actions #4

Updated by Loïc Dachary about 7 years ago

  • Copied to Backport #19049: kraken: multisite: some yields in RGWMetaSyncShardCR::full_sync() resume in incremental_sync() added
Actions #5

Updated by Nathan Cutler almost 7 years ago

  • Status changed from Pending Backport to Resolved
Actions

Also available in: Atom PDF