]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
mgr/nfs: don't output any message
authorRamana Raja <rraja@redhat.com>
Sun, 10 Apr 2022 18:18:53 +0000 (14:18 -0400)
committerRamana Raja <rraja@redhat.com>
Tue, 12 Apr 2022 20:20:15 +0000 (16:20 -0400)
... when `nfs cluster create/rm` succeeds. This also fixes the
misleading output message that said NFS cluster creation or
removal was complete when the `nfs cluster create/rm` command
returned. The `nfs cluster create/rm` command triggers the
asynchronous creation/removal of the NFS daemons and ingress
service of the cluster.

Fixes: https://tracker.ceph.com/issues/55299
Signed-off-by: Ramana Raja <rraja@redhat.com>
src/pybind/mgr/nfs/cluster.py

index 68980cb9cd8ad8b6f9f353d0d4a6719e4455f71b..4ddc750044eeb3b77cfc996997a55a346c4ddd51 100644 (file)
@@ -121,7 +121,7 @@ class NFSCluster:
 
             if cluster_id not in available_clusters(self.mgr):
                 self._call_orch_apply_nfs(cluster_id, placement, virtual_ip, port)
-                return 0, "NFS Cluster Created Successfully", ""
+                return 0, "", ""
             return 0, "", f"{cluster_id} cluster already exists"
         except Exception as e:
             return exception_handler(e, f"NFS Cluster {cluster_id} could not be created")
@@ -136,7 +136,7 @@ class NFSCluster:
                 completion = self.mgr.remove_service('nfs.' + cluster_id)
                 orchestrator.raise_if_exception(completion)
                 self.delete_config_obj(cluster_id)
-                return 0, "NFS Cluster Deleted Successfully", ""
+                return 0, "", ""
             return 0, "", "Cluster does not exist"
         except Exception as e:
             return exception_handler(e, f"Failed to delete NFS Cluster {cluster_id}")