delete not only running app (#169)

giulio93 · web-flow · commit 5b1dbf2838f9 · 2025-12-19T09:34:21.000+01:00
diff --git a/internal/orchestrator/orchestrator.go b/internal/orchestrator/orchestrator.go
@@ -897,15 +897,9 @@ func CloneApp(
 
 func DeleteApp(ctx context.Context, dockerClient command.Cli, app app.ArduinoApp) error{
 
-	runningApp, err := getRunningApp(ctx, dockerClient.Client())
-	if err != nil{
-		return err
-	}
-	if runningApp != nil && runningApp.FullPath.EqualsTo(app.FullPath){
-		// We try to remove docker related resources at best effort
-		for range StopAndDestroyApp(ctx, dockerClient, app){
-			// just consume the iterator
-		}
+	// We try to remove docker related resources at best effort
+	for range StopAndDestroyApp(ctx, dockerClient, app){
+		// just consume the iterator
 	}
 
 	return app.FullPath.RemoveAll()

-Original file line number
+Diff line change
 funcDeleteApp(ctx context.Context, dockerClient command.Cli, app app.ArduinoApp) error{
 -runningApp, err:=getRunningApp(ctx, dockerClient.Client())
 -iferr!=nil{
 -returnerr
 - }
 -ifrunningApp!=nil&&runningApp.FullPath.EqualsTo(app.FullPath){
 -// We try to remove docker related resources at best effort
 -forrangeStopAndDestroyApp(ctx, dockerClient, app){
 -// just consume the iterator
 - }
 +// We try to remove docker related resources at best effort
 +forrangeStopAndDestroyApp(ctx, dockerClient, app){
 +// just consume the iterator
+ }
 returnapp.FullPath.RemoveAll()