Project

General

Profile

Actions

Bug #36379

closed

mds: cache drop command only requires read caps

Added by Patrick Donnelly over 5 years ago. Updated over 5 years ago.

Status:
Rejected
Priority:
High
Category:
-
Target version:
% Done:

0%

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

Description

https://github.com/ceph/ceph/blob/a6a2f395b7e5f9fd867c0ba83a9e6525485a575d/src/mds/MDSDaemon.cc#L704-L707

This should be "rw" due to its potential impact on performance. Credit to John Spray for catching the issue.


Related issues 1 (0 open1 closed)

Related to CephFS - Cleanup #36380: mds: remove cap requirement on ceph tell commandsResolvedPatrick Donnelly10/10/2018

Actions
Actions #1

Updated by Patrick Donnelly over 5 years ago

  • Status changed from New to Rejected

Nevermind, turns out MDSDaemon ignores the command caps and checks client has "all":

https://github.com/ceph/ceph/blob/a6a2f395b7e5f9fd867c0ba83a9e6525485a575d/src/mds/MDSDaemon.cc#L613

Actions #2

Updated by Patrick Donnelly over 5 years ago

  • Related to Cleanup #36380: mds: remove cap requirement on ceph tell commands added
Actions

Also available in: Atom PDF