Project

General

Profile

Actions
Copy link

Bug #23855

closed

mds: MClientCaps should carry inode's dirstat

Added by Zheng Yan almost 6 years ago. Updated almost 6 years ago.

Status:
Resolved
Priority:
Urgent
Assignee:
Zheng Yan
Category:
Correctness/Safety
Target version:
Ceph - v13.2.0
% Done:

0%

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

Description

inode's dirstat gets updated by request reply, but not by cap message. this is problematic.

For example:
...
MDS modifies a directory
MDS issues CEPH_STAT_CAP_INODE_ALL to client
...

user stat(2) the directory
cephfs client finds that it has all shared caps locally, it return dirstat.size() as directory size.

Related issues 1 (0 open — 1 closed)

Copied to CephFS - Backport #24050: luminous: mds: MClientCaps should carry inode's dirstatResolvedPrashant DActions
Actions
Copy link
 #1

Updated by Patrick Donnelly almost 6 years ago

  • Subject changed from MClientCaps should carry inode's dirstat to mds: MClientCaps should carry inode's dirstat
  • Category set to Correctness/Safety
  • Assignee set to Zheng Yan
  • Priority changed from Normal to Urgent
  • Target version set to v13.0.0
  • Source set to Development
  • Backport set to luminous
  • Component(FS) MDS added
Actions
Copy link
 #2

Updated by Zheng Yan almost 6 years ago

  • Status changed from New to Fix Under Review
Actions
Copy link
 #3

Updated by Patrick Donnelly almost 6 years ago

  • Target version changed from v13.0.0 to v14.0.0
  • Backport changed from luminous to mimic,luminous
Actions
Copy link
 #4

Updated by Patrick Donnelly almost 6 years ago

  • Target version changed from v14.0.0 to v13.2.0
  • Backport changed from mimic,luminous to luminous
Actions
Copy link
 #5

Updated by Patrick Donnelly almost 6 years ago

  • Status changed from Fix Under Review to Pending Backport
Actions
Copy link
 #6

Updated by Nathan Cutler almost 6 years ago

  • Copied to Backport #24050: luminous: mds: MClientCaps should carry inode's dirstat added
Actions
Copy link
 #7

Updated by Nathan Cutler almost 6 years ago

  • Backport changed from luminous to luminous, mimic

Testing; will revert

Actions
Copy link
 #9

Updated by Nathan Cutler almost 6 years ago

  • Backport changed from luminous, mimic to luminous
Actions
Copy link
 #10

Updated by Nathan Cutler almost 6 years ago

  • Status changed from Pending Backport to Resolved
Actions
Copy link

Also available in: Atom PDF