fix(striker-ui-api): wrap missed job calls in try-catch

striker-ui-api/src/lib/request_handlers/host/updateHost.ts

@@ -4,7 +4,7 @@ import { LOCAL } from '../../consts/LOCAL';
 import SERVER_PATHS from '../../consts/SERVER_PATHS';
 
 import { job } from '../../accessModule';
-import { stdout } from '../../shell';
+import { stderr, stdout } from '../../shell';
 
 export const updateHost: RequestHandler<
   { hostUUID: string },
@@ -25,14 +25,22 @@ export const updateHost: RequestHandler<
   if (isEnableInstallTarget !== undefined) {
     const task = isEnableInstallTarget ? 'enable' : 'disable';
 
-    job({
+    try {
-      file: __filename,
+      job({
-      job_command: `${SERVER_PATHS.usr.sbin['striker-manage-install-target'].self} --${task}`,
+        file: __filename,
-      job_description: 'job_0016',
+        job_command: `${SERVER_PATHS.usr.sbin['striker-manage-install-target'].self} --${task}`,
-      job_host_uuid: hostUUID,
+        job_description: 'job_0016',
-      job_name: `install-target::${task}`,
+        job_host_uuid: hostUUID,
-      job_title: 'job_0015',
+        job_name: `install-target::${task}`,
-    });
+        job_title: 'job_0015',
+      });
+    } catch (subError) {
+      stderr(`Failed to ${task} install target; CAUSE: ${subError}`);
+
+      response.status(500).send();
+
+      return;
+    }
   }
 
   response.status(200).send();

striker-ui-api/src/lib/request_handlers/server/createServer.ts

@@ -6,10 +6,10 @@ import { REP_INTEGER, REP_UUID } from '../../consts/REG_EXP_PATTERNS';
 import SERVER_PATHS from '../../consts/SERVER_PATHS';
 
 import { dbQuery, job } from '../../accessModule';
+import { stderr, stdout } from '../../shell';
 
 export const createServer: RequestHandler = ({ body }, response) => {
-  console.log('Creating server.');
+  stdout(`Creating server.\n${JSON.stringify(body, null, 2)}`);
-  console.dir(body, { depth: null });
 
   const {
     serverName,
@@ -81,7 +81,7 @@ export const createServer: RequestHandler = ({ body }, response) => {
       `Data anvil UUID must be a valid UUID; got [${dataAnvilUUID}].`,
     );
   } catch (assertError) {
-    console.log(
+    stdout(
       `Failed to assert value when trying to provision a server; CAUSE: ${assertError}.`,
     );
 
@@ -99,7 +99,7 @@ storage_size=${dataStorageSize}
 install_iso=${dataInstallISO}
 driver_iso=${dataDriverISO}`;
 
-  console.log(`provisionServerJobData: [${provisionServerJobData}]`);
+  stdout(`provisionServerJobData=[${provisionServerJobData}]`);
 
   const [[provisionServerJobHostUUID]] = dbQuery(
     `SELECT
@@ -134,17 +134,25 @@ driver_iso=${dataDriverISO}`;
           ON pri_hos.phl = nod_1.phr;`,
   ).stdout;
 
-  console.log(`provisionServerJobHostUUID: [${provisionServerJobHostUUID}]`);
+  stdout(`provisionServerJobHostUUID=[${provisionServerJobHostUUID}]`);
-
+
-  job({
+  try {
-    file: __filename,
+    job({
-    job_command: SERVER_PATHS.usr.sbin['anvil-provision-server'].self,
+      file: __filename,
-    job_data: provisionServerJobData,
+      job_command: SERVER_PATHS.usr.sbin['anvil-provision-server'].self,
-    job_name: 'server:provision',
+      job_data: provisionServerJobData,
-    job_title: 'job_0147',
+      job_name: 'server:provision',
-    job_description: 'job_0148',
+      job_title: 'job_0147',
-    job_host_uuid: provisionServerJobHostUUID,
+      job_description: 'job_0148',
-  });
+      job_host_uuid: provisionServerJobHostUUID,
+    });
+  } catch (subError) {
+    stderr(`Failed to provision server; CAUSE: ${subError}`);
+
+    response.status(500).send();
+
+    return;
+  }
 
   response.status(202).send();
 };