mgr/volumes: Make cluster id option mandatory in nfs create export

Signed-off-by: Varsha Rao <[email protected]>
scabrero · Apr 8, 2020 · 25642bc · 25642bc
1 parent a84d4b0
commit 25642bc
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/doc/cephfs/fs-nfs-exports.rst b/doc/cephfs/fs-nfs-exports.rst
@@ -35,7 +35,7 @@ Create CephFS Export
 
 .. code:: bash
 
-    $ ceph nfs export create <type=cephfs> <fsname> <binding> [--readonly] [--path=/path/in/cephfs] [--attach=<clusterid>]
+    $ ceph nfs export create <type=cephfs> <fsname> <binding> <clusterid> [--readonly] [--path=/path/in/cephfs]
 
 It creates export rados objects containing the export block. Here binding is
 the pseudo root name and type is export type. Currently only CephFS is

diff --git a/src/pybind/mgr/volumes/module.py b/src/pybind/mgr/volumes/module.py
@@ -220,9 +220,9 @@ class Module(orchestrator.OrchestratorClientMixin, MgrModule):
             'name=type,type=CephString '
             'name=fsname,type=CephString '
             'name=binding,type=CephString '
+            'name=attach,type=CephString '
             'name=readonly,type=CephBool,req=false '
-            'name=path,type=CephString,req=false '
-            'name=attach,type=CephString,req=false ',
+            'name=path,type=CephString,req=false ',
             'desc': "Create a cephfs export",
             'perm': 'rw'
         },
@@ -409,7 +409,7 @@ def _cmd_nfs_export_create(self, inbuf, cmd):
         #TODO Extend export creation for rgw.
         return self.fs_export.create_export(export_type=cmd['type'], fs_name=cmd['fsname'],
                 pseudo_path=cmd['binding'], read_only=cmd.get('readonly', False),
-                path=cmd.get('path', '/'), cluster_id=cmd.get('attach','None'))
+                path=cmd.get('path', '/'), cluster_id=cmd.get('attach'))
 
     def _cmd_fs_nfs_export_delete(self, inbuf, cmd):
             return self.fs_export.delete_export(cmd['export_id'])

diff --git a/src/vstart.sh b/src/vstart.sh
@@ -1136,7 +1136,7 @@ start_ganesha() {
         pid file = $ganesha_dir/ganesha.pid
 EOF
 
-        prun ceph_adm nfs export create cephfs "a" "/cephfs" --attach=$name
+        prun ceph_adm nfs export create cephfs "a" "/cephfs" $name
         prun ganesha-rados-grace -p $pool_name -n $namespace add $name
         prun ganesha-rados-grace -p $pool_name -n $namespace