Project

General

Profile

Actions
Copy link

Bug #19816

closed

multisite: set_latest_epoch not atomic

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

Status:
Resolved
Priority:
High
Assignee:
Casey Bodley
Target version:
-
% Done:

0%

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

Description

see comment in rgw_period_puller.cc:

    // XXX: if this is a newer epoch, we should overwrite the existing
    // latest_epoch. but there's no way to do that atomically
    bool exclusive = true;
    r = period.set_latest_epoch(period.get_epoch(), exclusive);

Related issues 1 (0 open — 1 closed)

Related to rgw - Bug #19817: multisite: RGWPeriodPuller does not call RGWPeriod::reflect() on new periodResolvedCasey Bodley05/01/2017

Actions
Actions
Copy link
 #1

Updated by Casey Bodley almost 7 years ago

  • Status changed from New to Fix Under Review
  • Backport deleted (jewel kraken)

https://github.com/ceph/ceph/pull/14915

added backport tags to related issue http://tracker.ceph.com/issues/19817, removing them from this one

Actions
Copy link
 #2

Updated by Casey Bodley almost 7 years ago

  • Related to Bug #19817: multisite: RGWPeriodPuller does not call RGWPeriod::reflect() on new period added
Actions
Copy link
 #3

Updated by Yehuda Sadeh almost 7 years ago

  • Priority changed from Normal to High
Actions
Copy link
 #4

Updated by Yuri Weinstein almost 7 years ago

Casey Bodley wrote:

https://github.com/ceph/ceph/pull/14915

added backport tags to related issue http://tracker.ceph.com/issues/19817, removing them from this one

merged

Actions
Copy link
 #5

Updated by Casey Bodley almost 7 years ago

  • Status changed from Fix Under Review to Resolved
Actions
Copy link

Also available in: Atom PDF