Project

General

Profile

Bug #1117

mds: rename rollback broken on slaves during replay

Added by Greg Farnum almost 13 years ago. Updated over 10 years ago.

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

0%

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

Description

Best I can tell it's just busted. The rollback object contains all the dentries and inodes, but on a slave it's entirely probable that most of them won't exist during replay due to the strict auth requirements. And it asserts to make sure it has them in the cache.

This will need testing and going over at some point, but should be regarded as fairly low priority right now (as it requires a [rare] rollback to coincide with a dying MDS).

History

#1 Updated by Sage Weil almost 13 years ago

  • Target version set to v0.31

#2 Updated by Greg Farnum almost 13 years ago

It seems to also be broken on the master -- I've been testing cross-MDS rename ops and of course you see a lot of rollbacks if the slave crashes in the middle of the rename! The one I'm seeing so far is a segfault when it tries to print out a non-existent cache object.

#3 Updated by Sage Weil almost 13 years ago

  • Target version changed from v0.31 to v0.32

#4 Updated by Sage Weil over 12 years ago

  • Target version changed from v0.32 to v0.33
  • translation missing: en.field_position set to 1

#5 Updated by Sage Weil over 12 years ago

  • translation missing: en.field_position deleted (44)
  • translation missing: en.field_position set to 1
  • translation missing: en.field_position changed from 1 to 774

#6 Updated by Sage Weil over 12 years ago

  • Target version changed from v0.33 to v0.34

#7 Updated by Sage Weil over 12 years ago

  • Target version changed from v0.34 to v0.35
  • translation missing: en.field_position deleted (788)
  • translation missing: en.field_position set to 1

#8 Updated by Sage Weil over 12 years ago

  • translation missing: en.field_position deleted (17)
  • translation missing: en.field_position set to 24

#9 Updated by Sage Weil over 12 years ago

  • Subject changed from rename rollback broken on slaves during replay to mds: rename rollback broken on slaves during replay
  • translation missing: en.field_position deleted (24)
  • translation missing: en.field_position set to 24

#10 Updated by Sage Weil over 12 years ago

  • Target version changed from v0.35 to v0.36
  • translation missing: en.field_position deleted (61)
  • translation missing: en.field_position set to 17

#11 Updated by Sage Weil over 12 years ago

  • Target version deleted (v0.36)
  • translation missing: en.field_position deleted (19)
  • translation missing: en.field_position set to 242

#12 Updated by Sage Weil over 11 years ago

  • Project changed from Ceph to CephFS
  • Category deleted (1)

#13 Updated by Greg Farnum about 11 years ago

  • Priority changed from Normal to Low

De-prioritizing multi-mds issues for now.

#14 Updated by Zheng Yan over 10 years ago

  • Status changed from New to Resolved

#15 Updated by Greg Farnum over 10 years ago

Which patch[es]?

#16 Updated by Zheng Yan over 10 years ago

by commit 844cd46c77274ee7726ded8bf0d83e7f586da00e

Also available in: Atom PDF