Actions
Bug #62626
openmgr/nfs: include pseudo in JSON output when nfs export apply -i fails
Status:
Pending Backport
Priority:
Normal
Assignee:
Category:
Correctness/Safety
Target version:
% Done:
0%
Source:
Development
Tags:
backport_processed
Backport:
reef,quincy
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
Currently, when export update fails, this is the reponse:
[
{
"msg": "Failed to apply export: export FSAL user_id must be 'nfs.my-nfs.1'",
"state": "error"
},
{
"msg": "Failed to apply export: export FSAL user_id must be 'nfs.my-nfs.2'",
"state": "error"
}
]
in case of multiple export update failure, it is tedious to figure out failure for respective exports, this can be fixed with if we include export_id with and also errno to help understand the failure behind unsuccessful export update i.e. like this:
[
{
"errno": 22,
"export_id": 1,
"msg": "Failed to apply export: export FSAL user_id must be 'nfs.my-nfs.1'",
"state": "error"
},
{
"errno": 22,
"export_id": 2,
"msg": "Failed to apply export: export FSAL user_id must be 'nfs.my-nfs.2'",
"state": "error"
}
]
also add test case that tries to update multiple export(this is essential not only this tracker but for test suite as well because currently there is no test case that tests multiple exports manipulation)
Actions