Project

General

Profile

Actions

Feature #47264

closed

"fs authorize" subcommand should work for multiple FSs too

Added by Rishabh Dave over 5 years ago. Updated 8 months ago.

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

0%

Source:
Development
Backport:
Reviewed:
Affected Versions:
Component(FS):
Common/Protocol, MDSMonitor
Labels (FS):
Pull request ID:
Tags (freeform):
Fixed In:
v18.0.0-5932-g849fbd1d5b
Released In:
v19.2.0~1673
Upkeep Timestamp:
2025-07-14T19:31:23+00:00

Description

Currently assigning caps for a second FS to an already existing client (which holds caps for a different FS already) using "ceph fs authorize" subcommand is not possible -

$ ./bin/ceph fs ls
name: a, metadata pool: cephfs.a.meta, data pools: [cephfs.a.data ]
name: cephfs2, metadata pool: cephfs2_meta, data pools: [cephfs2_data ]
$ ./bin/ceph fs authorize a client.someuser / rw
[client.someuser]
    key = AQAjeUNfcvezHhAAzFTLqmpzZRqgEV5bRReChw==
$ ./bin/ceph fs authorize cephfs2 client.someuser / rw
Error EINVAL: client.someuser already has fs capabilities that differ from those supplied. To generate a new auth key for client.someuser, first remove client.someuser from configuration files, execute 'ceph auth rm client.someuser', then execute this command again.
$

I think it would be very convenient for CephFS users to be able to assign caps for multiple FSs to already existing clients using this subcommand. Ticket #15070 (PR #32581) already adds the ability for a client to have caps for multiple FSs, we just need to modify behaviour for "fs authorize" subcommand to get this done.

Also, IMO, it would be nice to have the reverse of the behaviour proposed above: removing caps only for a certain FS for a client. We can add a new subcommand "fs deauthorize" the syntax of which would be same as that of "fs authorize".


Related issues 5 (1 open4 closed)

Related to CephFS - Feature #15070: mon: client: multifs: auth caps on client->mon connections to limit their access to MDSMaps by FSCIDResolvedRishabh Dave

Actions
Related to CephFS - Feature #56428: add command "fs deauthorize"NewRishabh Dave

Actions
Related to CephFS - Bug #64182: mds: "fs authorize" update MDS caps only if it is last one to be consideredResolvedRishabh Dave

Actions
Related to CephFS - Bug #64417: mds: mds caps can't be updated to add root_squashResolvedRishabh Dave

Actions
Related to CephFS - Bug #64127: mds: passing multiple caps to "fs authorize" cmd causes MON to crashResolvedRishabh Dave

Actions
Actions #1

Updated by Rishabh Dave over 5 years ago

  • Related to Feature #15070: mon: client: multifs: auth caps on client->mon connections to limit their access to MDSMaps by FSCID added
Actions #2

Updated by Patrick Donnelly over 5 years ago

  • Target version set to v16.0.0
  • Source set to Development
Actions #3

Updated by Patrick Donnelly about 5 years ago

  • Target version changed from v16.0.0 to v17.0.0
Actions #4

Updated by Rishabh Dave almost 5 years ago

  • Status changed from New to In Progress
Actions #5

Updated by Rishabh Dave almost 5 years ago

  • Status changed from In Progress to Fix Under Review
  • Pull request ID set to 41779
Actions #6

Updated by Rishabh Dave over 3 years ago

Actions #7

Updated by Patrick Donnelly over 3 years ago

  • Target version deleted (v17.0.0)
Actions #8

Updated by Rishabh Dave over 2 years ago

  • Status changed from Fix Under Review to Resolved
Actions #9

Updated by Rishabh Dave over 2 years ago

Apparently, release note was added in wrong part of PendingReleaseNote (or it was added to right part but it was moved later). This PR moves it to right part - https://tracker.ceph.com/issues/47264.

Actions #10

Updated by Venky Shankar about 2 years ago

  • Related to Bug #64182: mds: "fs authorize" update MDS caps only if it is last one to be considered added
Actions #11

Updated by Patrick Donnelly almost 2 years ago

  • Target version set to v20.0.0
Actions #12

Updated by Rishabh Dave over 1 year ago

  • Related to Bug #64417: mds: mds caps can't be updated to add root_squash added
Actions #13

Updated by Rishabh Dave over 1 year ago

  • Related to Bug #64127: mds: passing multiple caps to "fs authorize" cmd causes MON to crash added
Actions #14

Updated by Upkeep Bot 8 months ago

  • Merge Commit set to 849fbd1d5b1d232e29feeb23f7c93e767569c31f
  • Fixed In set to v18.0.0-5932-g849fbd1d5b
  • Released In set to v19.2.0~1673
  • Upkeep Timestamp set to 2025-07-14T19:31:23+00:00
Actions

Also available in: Atom PDF