fix: remove stale devbox container on restart (#1043)

Signed-off-by: Kevin Su <pingsutw@apache.org>

Author: pingsutw

src/flyte/cli/_delete.py

@@ -80,9 +80,8 @@ def devbox(volume: bool):
     Stop and remove the local Flyte devbox cluster container.
     """
     console = common.get_console()
-    result = subprocess.run(["docker", "stop", "flyte-devbox"], capture_output=True, check=False)
+    result = subprocess.run(["docker", "rm", "-f", "flyte-devbox"], capture_output=True, check=False)
     if result.returncode == 0:
-        subprocess.run(["docker", "wait", "flyte-devbox"], capture_output=True, check=False)
         console.print("[green]Devbox cluster stopped.[/green]")
     else:
         console.print("[yellow]Devbox cluster is not running.[/yellow]")

src/flyte/cli/_devbox.py

@@ -254,7 +254,7 @@ def launch_devbox(image_name: str, is_dev_mode: bool, gpu: bool = False, log_for
         console.print("[yellow]Flyte devbox cluster is already running.[/yellow]")
         if not click.confirm("Do you want to delete the existing devbox cluster and start a new one?"):
             return
-        subprocess.run(["docker", "stop", _CONTAINER_NAME], check=True, capture_output=True)
+    subprocess.run(["docker", "rm", "-f", _CONTAINER_NAME], check=False, capture_output=True)
 
     _KUBE_DIR.mkdir(parents=True, exist_ok=True)
     # This step makes sure that we always used the latest k3s kubeconfig file