Fix poc shutdown

docs/real_world_fl/migrating_to_flare_api.rst

@@ -106,9 +106,6 @@ and the new way with FLARE API.
     restart,restart,2.4.0,
     shutdown,shutdown,2.4.0,
     set_timeout,set_timeout,2.4.0,changed to session-based
-    list_sp,list_sp,2.4.0,
-    get_active_sp,get_active_sp,2.4.0,
-    promote_sp,promote_sp,2.4.0,
     get_available_apps_to_upload,get_available_apps_to_upload,2.4.0,
     shutdown_system,shutdown_system,2.4.0,
     ls_target,ls_target,2.4.0,

docs/real_world_fl/operation.rst

@@ -58,19 +58,8 @@ commands shown as examples of how they may be run with a description.
     ,``restart server``,Restarts the server. Clients will also be restarted. Note that the admin client will need to log in again after the server restarts.
     shutdown,``shutdown client``,Shuts down all of the clients. Individual clients can be shutdown by specifying *clientname*. Please note that this may not be instant but may take time for the command to take effect.
     ,``shutdown server``,Shuts down the active server. Clients must be shut down first before the server is shut down. Note this will not shut down the Overseer or other SPs.
-    get_active_sp,``get_active_sp``,Get information on the active SP (service provider or FL server).
-    list_sp,``list_sp``,Get data from last heartbeat of the active and available SP endpoint information.
-    promote_sp,``promote_sp sp_end_point``,promote a specified SP to become the active SP (promote_sp example1.com:8002:8003)
     shutdown_system,``shutdown_system``,Shut down entire system by setting the system state to shutdown through the overseer
 
-
-.. note::
-
-   The commands ``promote_sp`` and ``shutdown_system`` both go to the Overseer and have a different mechanism of
-   authorization than the other commands sent to the FL server. The Overseer keeps track of a list of privileged users,
-   configured to be admin users with the role of "super". Only users owning certificates whose cn is in the privileged
-   user list can call these commands.
-
 .. tip::
 
    Outputs of any command can be redirected into a file by using the greater-than symbol ">", however there must be no

nvflare/tool/poc/poc_commands.py

@@ -689,20 +689,20 @@ def _stop_poc(poc_workspace: str, excluded=None, services_list=None):
 
     p_size = len(services_list)
     if p_size == 0 or service_config[SC.FLARE_SERVER] in services_list:
-        print("start shutdown NVFLARE")
+        print("start shutdown NVFLARE using FLARE API")
         shutdown_system(prod_dir, username=service_config[SC.FLARE_PROJ_ADMIN])
     else:
-        print(f"start shutdown {services_list}")
-
-    _run_poc(
-        SC.CMD_STOP,
-        poc_workspace,
-        gpu_ids,
-        service_config,
-        project_config,
-        excluded=excluded,
-        services_list=services_list,
-    )
+        print(f"start shutdown {services_list} using stop_fl.sh")
+
+        _run_poc(
+            SC.CMD_STOP,
+            poc_workspace,
+            gpu_ids,
+            service_config,
+            project_config,
+            excluded=excluded,
+            services_list=services_list,
+        )
 
 
 def _get_clients(service_commands: list, service_config) -> List[str]: