Project

General

Profile

Actions

Backport #16794

closed

jewel: multisite bucket sync doesn't retry objects that it fails to fetch

Added by Nathan Cutler almost 8 years ago. Updated over 7 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Target version:
Release:
jewel
Pull request ID:
Crash signature (v1):
Crash signature (v2):


Related issues 1 (0 open1 closed)

Copied from rgw - Bug #16742: multisite bucket sync doesn't retry objects that it fails to fetchResolved07/19/2016

Actions
Actions #1

Updated by Nathan Cutler almost 8 years ago

  • Copied from Bug #16742: multisite bucket sync doesn't retry objects that it fails to fetch added
Actions #2

Updated by Loïc Dachary over 7 years ago

https://github.com/ceph/ceph/pull/10355/commits/aacc31f72cef6f151459e4a0543850edeeaf1938 does not apply cleanly and is non trivial to resolve. There seem to be a need for another backport, to change

collect(&ret);

into
collect(&ret, lease_stack);

and all it entails.

Actions #3

Updated by Loïc Dachary over 7 years ago

  • Status changed from New to Need More Info
  • Assignee set to Yehuda Sadeh

@Yehuda Sadeh How do you suggest we handle this backport ? Resolving conflicts or cherry-picking more commits to avoid conflicts ?

Actions #4

Updated by Yehuda Sadeh over 7 years ago

  • Assignee changed from Yehuda Sadeh to Casey Bodley

sounds like we need to pull in more commits

Actions #5

Updated by Casey Bodley over 7 years ago

  • Status changed from Need More Info to New
  • Assignee deleted (Casey Bodley)

the missing commit had already been backported. i cherry-picked the commits cleanly here: https://github.com/ceph/ceph/pull/11021

Actions #6

Updated by Nathan Cutler over 7 years ago

  • Description updated (diff)
  • Status changed from New to In Progress
  • Assignee set to Nathan Cutler
Actions #7

Updated by Yehuda Sadeh over 7 years ago

  • Status changed from In Progress to Resolved
Actions #8

Updated by Loïc Dachary over 7 years ago

  • Target version set to v10.2.4
Actions

Also available in: Atom PDF