Actions
Bug #58228
openmgr/nfs: disallow non-existent paths when creating export
Status:
Pending Backport
Priority:
Normal
Assignee:
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.
Updated by Dhairya Parmar over 1 year ago
- Status changed from New to Fix Under Review
Updated by Venky Shankar about 1 year 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
Updated by Backport Bot about 1 year ago
- Copied to Backport #59250: quincy: mgr/nfs: disallow non-existent paths when creating export added
Updated by Backport Bot about 1 year ago
- Copied to Backport #59251: reef: mgr/nfs: disallow non-existent paths when creating export added
Updated by Backport Bot about 1 year ago
- Copied to Backport #59252: pacific: mgr/nfs: disallow non-existent paths when creating export added
Updated by Venky Shankar 12 months ago
Dhairya, please pick up additional commits from https://github.com/ceph/ceph/pull/51005.
Updated by Dhairya Parmar 12 months ago
Venky Shankar wrote:
Dhairya, please pick up additional commits from https://github.com/ceph/ceph/pull/51005.
Alright
Updated by Dhairya Parmar 12 months ago
Dhairya Parmar wrote:
Venky Shankar wrote:
Dhairya, please pick up additional commits from https://github.com/ceph/ceph/pull/51005.
Alright
Done
Actions