Fixed: the automatic removal of deployments from engines that were previously not reachable does not remove the engines from the deployment information when the removal succeeds

Author: jjoderis

src/management-system-v2/lib/executions/deployment-server-actions.ts

@@ -28,6 +28,7 @@ import { MapNestedType, Prettify, truthyFilter } from '../typescript-utils';
 import { addDeployment, updateDeployment, getProcessDeployments } from '../data/deployment';
 import { revalidateTag } from 'next/cache';
 import { updateTaskInfo } from '../tasks/server-actions';
+import { reach } from 'yup';
 
 export async function deployProcess(
   definitionId: string,
@@ -366,9 +367,9 @@ export async function refetchDeployments() {
           fetchedVersion.machines = await asyncFilter(fetchedVersion.machines, async (m) => {
             try {
               await removeDeploymentFromMachines([m], d.version.processId);
-              return true;
-            } catch (err) {
               return false;
+            } catch (err) {
+              return true;
             }
           });
         }