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

Signed-off-by: Varsha Rao <varao@redhat.com>

diff --git a/doc/cephfs/fs-nfs-exports.rst b/doc/cephfs/fs-nfs-exports.rst
index aa9f69de0f90405d670c4ec248713622a7db8e19..f18bbbae170c30e1b651e5f47ebf62616742c130 100644 (file)
--- 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
index a5cbe960f7dc97e99798a4586214e6815545616e..a1ab40d1658cd45458a51528b11d03452287c93a 100644 (file)
--- 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 @@ class Module(orchestrator.OrchestratorClientMixin, MgrModule):
         #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
index 11db9555cd817e2267ee0c52f48af815ea711b5e..47e1f34f8984e943907004ccfcb3002eb925d439 100755 (executable)
--- 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