Project

General

Profile

Actions

Feature #3540

closed

mds: maintain per-file backpointers on first file object

Added by Sage Weil over 11 years ago. Updated about 11 years ago.

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

0%

Source:
Tags:
Backport:
Reviewed:
Affected Versions:
Component(FS):
Labels (FS):
Pull request ID:

Description

Add backtraces to each file object, as described at http://www.spinics.net/lists/ceph-devel/msg11872.html. This ticket includes the MDS asynchronously updating backtraces on creates and renames, as well as the "sentinel object" infrastructure, but not the lookup infrastructure.

For this initial implementation, the MDS should add the backtrace on file create (so we end up with a sentinel for all objects whose pool gets changed).

Actions #1

Updated by Sage Weil over 11 years ago

  • Translation missing: en.field_position deleted (1)
  • Translation missing: en.field_position set to 15
Actions #2

Updated by Sage Weil over 11 years ago

  • Translation missing: en.field_story_points set to 13
  • Translation missing: en.field_position deleted (16)
  • Translation missing: en.field_position set to 16
Actions #3

Updated by Greg Farnum over 11 years ago

Do we have any kind of design for this? We've talked about it some and it's conceptually simple, but splitting up the responsibility between client and MDS for doing this, and then making sure we clearly define the consistency guarantees and follow them, is going to take some work.

Actions #4

Updated by Sage Weil over 11 years ago

  • Status changed from New to 12

Greg Farnum wrote:

Do we have any kind of design for this? We've talked about it some and it's conceptually simple, but splitting up the responsibility between client and MDS for doing this, and then making sure we clearly define the consistency guarantees and follow them, is going to take some work.

No concrete design yet. We need to discuss this as a team before implementing anything.

My initial thought is to have the mds to this, but if we have the client piggyback it on writeback that would be more efficient. Maybe that's v2.

Actions #5

Updated by Greg Farnum over 11 years ago

  • Description updated (diff)

We have a design now!

Actions #6

Updated by Greg Farnum over 11 years ago

  • Translation missing: en.field_story_points deleted (13)
  • Translation missing: en.field_position deleted (16)
  • Translation missing: en.field_position set to 16
Actions #7

Updated by Sage Weil over 11 years ago

  • Translation missing: en.field_story_points set to 8
  • Translation missing: en.field_position deleted (17)
  • Translation missing: en.field_position set to 6
Actions #8

Updated by Sage Weil over 11 years ago

  • Translation missing: en.field_position deleted (6)
  • Translation missing: en.field_position set to 2
Actions #9

Updated by Sage Weil over 11 years ago

  • Target version set to v0.57c
  • Translation missing: en.field_position deleted (2)
  • Translation missing: en.field_position set to 2
Actions #10

Updated by Ian Colle over 11 years ago

  • Assignee set to Sam Lang
Actions #11

Updated by Sam Lang about 11 years ago

  • Status changed from 12 to In Progress
Actions #12

Updated by Sam Lang about 11 years ago

  • Status changed from In Progress to Fix Under Review

Initial implementation in wip-bt. Needs review.

Actions #13

Updated by Greg Farnum about 11 years ago

  • Status changed from Fix Under Review to In Progress

The initial review happened last week; Sam has some updates for the rename and sentinel object infrastructure now but they aren't ready for review.

Actions #14

Updated by Sam Lang about 11 years ago

  • Status changed from In Progress to 7

I've pushed the additional changes for rename and sentinel to the wip-bt2 branch. Those bits are still untested, but can probably be reviewed in the mean-time...

Actions #15

Updated by Ian Colle about 11 years ago

  • Target version changed from v0.57c to v0.58
  • Translation missing: en.field_position deleted (5)
  • Translation missing: en.field_position set to 2
Actions #16

Updated by Greg Farnum about 11 years ago

Comments on Github for this.

Actions #17

Updated by Greg Farnum about 11 years ago

  • Target version changed from v0.58 to v0.59
Actions #18

Updated by Greg Farnum about 11 years ago

  • Target version changed from v0.59 to v0.60
Actions #19

Updated by Ian Colle about 11 years ago

  • Status changed from 7 to Resolved

8b798867731d298c05d9f93b0c207a541d2b5e90 merged to master

Actions

Also available in: Atom PDF