An attempt to start an instance when no engine is available will now lead to a user readable error instead of silently failing

Author: jjoderis

src/management-system-v2/app/(dashboard)/[environmentId]/(automation)/executions/deployment-hook.ts

@@ -100,14 +100,14 @@ function useDeployment(definitionId: string, initialData?: DeployedProcessInfo)
   });
 
   const startInstance = async (versionId: string, variables: { [key: string]: any } = {}) => {
-    if (engines?.length) {
-      // TODO: in case of static deployment or different versions on different engines we will have
-      // to check if the engine can actually be used to start an instance
-      return await startInstanceOnMachine(definitionId, versionId, engines[0], variables, {
-        processInitiator: session?.user.id,
-        spaceIdOfProcessInitiator: space.spaceId,
-      });
-    }
+    if (!engines?.length) return userError('No fitting engine found');
+
+    // TODO: in case of static deployment or different versions on different engines we will have
+    // to check if the engine can actually be used to start an instance
+    return await startInstanceOnMachine(definitionId, versionId, engines[0], variables, {
+      processInitiator: session?.user.id,
+      spaceIdOfProcessInitiator: space.spaceId,
+    });
   };
 
   const activeStates = ['PAUSED', 'RUNNING', 'READY', 'DEPLOYMENT-WAITING', 'WAITING'];
@@ -171,7 +171,8 @@ function useDeployment(definitionId: string, initialData?: DeployedProcessInfo)
   }
 
   async function getStartForm(versionId: string) {
-    if (!engines) return;
+    if (!engines?.length) return userError('No fitting engine found');
+
     try {
       // TODO: in case of static deployment or different versions on different engines we will have
       // to check if the engine can actually be used to start an instance