Project

General

Profile

Actions
Copy link

Bug #50783

closed

mgr/nfs: cli is broken as cluster id and binding arguments are optional

Added by Varsha Rao almost 3 years ago. Updated almost 3 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Varsha Rao
Category:
-
Target version:
Ceph - v17.0.0
% Done:

0%

Source:
Development
Tags:
Backport:
pacific
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
mgr/nfs
Labels (FS):
NFS-cluster
Pull request ID:
41314
Crash signature (v1):
Crash signature (v2):

Description

In the following commands clusterid requirement is made optional which breaks cli.

nfs cluster config reset [<clusterid>]                                                                                 Reset NFS-Ganesha Config to default
nfs cluster create [<clusterid>] [<placement>]                                                                         Create an NFS Cluster
nfs cluster delete [<clusterid>]                                                                                       Removes an NFS Cluster (DEPRECATED)
nfs cluster rm [<clusterid>]                                                                                           Removes an NFS Cluster
nfs export ls [<clusterid>] [--detailed]                                                                               List exports of a NFS cluster

binding is required in the following commands

nfs export create cephfs <fsname> <clusterid> [<binding>] [--readonly] [<path>]                                        Create a cephfs export
nfs export delete <clusterid> [<binding>]                                                                              Delete a cephfs export (DEPRECATED)
nfs export get <clusterid> [<binding>]                                                                                 Fetch a export of a NFS cluster given the pseudo path/binding
nfs export rm <clusterid> [<binding>]                                                                                  Remove a cephfs export

This commit1 breaks the cli.

[1] https://github.com/ceph/ceph/commit/7c57d1634049484b3037d297e0955b14dcac9654#diff-12b7237c4297fd7ef5e2f4e41e0b6a5718710221829af6d5df8a35886345610f

Related issues 1 (0 open — 1 closed)

Copied to CephFS - Backport #50843: pacific: mgr/nfs: cli is broken as cluster id and binding arguments are optionalResolvedVarsha RaoActions
Actions
Copy link
 #1

Updated by Varsha Rao almost 3 years ago

  • Status changed from In Progress to Fix Under Review
  • Pull request ID set to 41314
Actions
Copy link
 #2

Updated by Patrick Donnelly almost 3 years ago

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

Updated by Backport Bot almost 3 years ago

  • Copied to Backport #50843: pacific: mgr/nfs: cli is broken as cluster id and binding arguments are optional added
Actions
Copy link
 #4

Updated by Loïc Dachary almost 3 years ago

  • Status changed from Pending Backport to Resolved

While running with --resolve-parent, the script "backport-create-issue" noticed that all backports of this issue are in status "Resolved" or "Rejected".

Actions
Copy link

Also available in: Atom PDF