Project

General

Profile

Bug #58228

mgr/nfs: disallow non-existent paths when creating export

Added by Venky Shankar about 1 year ago. Updated 10 months ago.

Status:
Pending Backport
Priority:
Normal
Category:
Administration/Usability
Target version:
% Done:

0%

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

Description

Right now the path isn't validated, so one can:

# ceph nfs export create cephfs cephfs testnfs /cephfs/d6 /non_existed_dir
{
    "bind": "/cephfs/d6",
    "fs": "cephfs",
    "path": "/non_existed_dir",
    "cluster": "testnfs",
    "mode": "RW" 
}

and the export will be happily created. Mounting the export would fail obviously, however, this can be checked when creating the export and informing the operator.


Related issues

Copied to CephFS - Backport #59250: quincy: mgr/nfs: disallow non-existent paths when creating export In Progress
Copied to CephFS - Backport #59251: reef: mgr/nfs: disallow non-existent paths when creating export Resolved
Copied to CephFS - Backport #59252: pacific: mgr/nfs: disallow non-existent paths when creating export Resolved

History

#1 Updated by Dhairya Parmar about 1 year ago

  • Pull request ID set to 49460

#2 Updated by Dhairya Parmar about 1 year ago

  • Status changed from New to Fix Under Review

#3 Updated by Venky Shankar 11 months ago

  • Status changed from Fix Under Review to Pending Backport
  • Target version changed from v18.0.0 to v19.0.0
  • Backport changed from pacific,quincy to reef,quincy,pacific

#4 Updated by Backport Bot 11 months ago

  • Copied to Backport #59250: quincy: mgr/nfs: disallow non-existent paths when creating export added

#5 Updated by Backport Bot 11 months ago

  • Copied to Backport #59251: reef: mgr/nfs: disallow non-existent paths when creating export added

#6 Updated by Backport Bot 11 months ago

  • Copied to Backport #59252: pacific: mgr/nfs: disallow non-existent paths when creating export added

#7 Updated by Backport Bot 11 months ago

  • Tags set to backport_processed

#8 Updated by Venky Shankar 10 months ago

Dhairya, please pick up additional commits from https://github.com/ceph/ceph/pull/51005.

#9 Updated by Dhairya Parmar 10 months ago

Venky Shankar wrote:

Dhairya, please pick up additional commits from https://github.com/ceph/ceph/pull/51005.

Alright

#10 Updated by Dhairya Parmar 10 months ago

Dhairya Parmar wrote:

Venky Shankar wrote:

Dhairya, please pick up additional commits from https://github.com/ceph/ceph/pull/51005.

Alright

Done

Also available in: Atom PDF