Project

General

Profile

Actions

Bug #5512

closed

mon: missing full osdmaps after sync

Added by Sage Weil almost 11 years ago. Updated almost 11 years ago.

Status:
Resolved
Priority:
Urgent
Assignee:
Category:
-
Target version:
-
% Done:

0%

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

Description

full osdmaps are occasionally part of paxos transactions. if one of these overlaps with a sync, the synced monitor won't generate and write out preceding full osdmaps.

next fix in wip-mon-fix-osdmap-sync, needs cuttlefish backport too. but first i need to make teuthology + scrub catch the existing bug.

Actions #1

Updated by Sage Weil almost 11 years ago

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

Updated by Sage Weil almost 11 years ago

  • Assignee set to Sage Weil
Actions #3

Updated by Sage Weil almost 11 years ago

  • Status changed from Pending Backport to Resolved
Actions #4

Updated by Sage Weil almost 11 years ago

  • Status changed from Resolved to Pending Backport
Actions #5

Updated by Sage Weil almost 11 years ago

digging into why the scrub error came up on cuttlefish; this has been backported.

Actions #6

Updated by Greg Farnum almost 11 years ago

Did we backport the update on commit versus round finish patch?

Actions #7

Updated by Sage Weil almost 11 years ago

Greg Farnum wrote:

Did we backport the update on commit versus round finish patch?

oh, that's right.. that's what this is.

there is 1 patch that addresses this, but it is immediately followed by 5 more that clean up all the surrounding code, and i'm nervous about taking it in isolation, or doing the whole series.

we could just disable the scrub on cuttlefish, or even just make it not scrub if paxos->is_updating(). this only affects scrub, AFAICS.

Actions #8

Updated by Sage Weil almost 11 years ago

  • Status changed from Pending Backport to Resolved
Actions

Also available in: Atom PDF