Update: Fail job if there are errors (#96)

razee-io · May 13, 2020 · 575ac9c · 575ac9c
1 parent 8023678
commit 575ac9c
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 2 deletions.
diff --git a/src/install.js b/src/install.js
@@ -30,6 +30,7 @@ const fs = require('fs-extra');
 const axios = require('axios');
 const handlebars = require('handlebars');
 
+var success = true;
 const argvNamespace = typeof (argv.n || argv.namespace) === 'string' ? argv.n || argv.namespace : 'razeedeploy';
 
 async function main() {
@@ -97,6 +98,7 @@ async function main() {
 
   let fileSource = typeof (argv.s || argv['file-source']) === 'string' ? argv.s || argv['file-source'] : 'https://github.com/razee-io';
   if (!validUrl.isUri(fileSource)) {
+    success = false;
     return log.error(`'${fileSource}' not a valid source url.`);
   } else if (fileSource.endsWith('/')) {
     fileSource = fileSource.replace(/\/+$/g, '');
@@ -177,13 +179,15 @@ async function main() {
         await crdRegistered('deploy.razee.io/v1alpha2', 'RemoteResource');
         await decomposeFile(autoUpdateJson);
       } catch (e) {
+        success = false;
         log.error(`${e}.. skipping autoUpdate`);
       }
     } else if (autoUpdate && !(installAll || resourcesObj.remoteresource.install)) {
       log.info('=========== Installing Auto-Update RemoteResource ===========');
       log.warn('RemoteResource CRD must be one of the installed resources in order to use autoUpdate. (ie. --rr -a).. Skipping autoUpdate');
     }
   } catch (e) {
+    success = false;
     log.error(e);
   }
 }
@@ -204,6 +208,7 @@ async function crdRegistered(apiVersion, kind, attempts = 5, backoffInterval = 5
     log.info(`Found ${apiVersion} ${kind}`);
     return krm;
   } else if (--attempts <= 0) {
+    success = false;
     throw Error(`Failed to find ${apiVersion} ${kind}`);
   } else {
     log.warn(`Did not find ${apiVersion} ${kind}.. attempts remaining: ${attempts}`);
@@ -253,9 +258,11 @@ async function decomposeFile(file, mode = 'replace') {
           await replace(krm, file);
         }
       } catch (e) {
+        success = false;
         log.error(e);
       }
     } else {
+      success = false;
       log.error(`KubeResourceMeta not found: { kind: ${kind}, apiVersion: ${apiVersion}, name: ${objectPath.get(file, 'metadata.name')}, namespace: ${objectPath.get(file, 'metadata.namespace')} } ... skipping`);
     }
   }
@@ -341,4 +348,9 @@ async function ensureExists(krm, file, options = {}) {
   return response;
 }
 
-main().catch(log.error);
+main().then(() => {
+  success === true ? process.exit(0) : process.exit(1);
+}).catch(e => {
+  log.error(e);
+  process.exit(1);
+});
diff --git a/src/remove.js b/src/remove.js
@@ -28,6 +28,7 @@ const fs = require('fs-extra');
 const axios = require('axios');
 const handlebars = require('handlebars');
 
+var success = true;
 const argvNamespace = typeof (argv.n || argv.namespace) === 'string' ? argv.n || argv.namespace : 'razeedeploy';
 
 async function main() {
@@ -77,6 +78,7 @@ async function main() {
 
   let fileSource = typeof (argv.s || argv['file-source']) === 'string' ? argv.s || argv['file-source'] : 'https://github.com/razee-io';
   if (!validUrl.isUri(fileSource)) {
+    success = false;
     return log.error(`'${fileSource}' not a valid source url.`);
   } else if (fileSource.endsWith('/')) {
     fileSource = fileSource.replace(/\/+$/g, '');
@@ -132,6 +134,7 @@ async function main() {
               await crdDeleted(objectPath.get(crd, 'metadata.name'), attempts, backoff);
             }
           } catch (e) {
+            success = false;
             log.error(`Timed out trying to safely clean up crd ${objectPath.get(crd, 'metadata.name')}.. use option '-f, --force' to force clean up (note: child resources wont be cleaned up)`);
           }
         }
@@ -152,6 +155,7 @@ async function main() {
     }
 
   } catch (e) {
+    success = false;
     log.error(e);
   }
 }
@@ -187,6 +191,7 @@ async function crdDeleted(name, attempts = 5, backoffInterval = 3750) {
     log.info(`Successfully deleted ${name}`);
     return;
   } else if (--attempts <= 0) {
+    success = false;
     throw Error(`Failed to delete ${name}`);
   } else {
     log.warn(`CRD ${name} not fully removed.. re-checking in: ${backoffInterval/1000} sec, attempts remaining: ${attempts}`);
@@ -246,9 +251,11 @@ async function deleteFile(file, force = false) {
       try {
         await deleteResource(krm, file, { force: force });
       } catch (e) {
+        success = false;
         log.error(e);
       }
     } else {
+      success = false;
       log.error(`KubeResourceMeta not found: { kind: ${kind}, apiVersion: ${apiVersion}, name: ${objectPath.get(file, 'metadata.name')}, namespace: ${objectPath.get(file, 'metadata.namespace')} } ... skipping`);
     }
   }
@@ -287,4 +294,9 @@ async function deleteResource(krm, file, options = {}) {
   }
 }
 
-main().catch(log.error);
+main().then(() => {
+  success === true ? process.exit(0) : process.exit(1);
+}).catch(e => {
+  log.error(e);
+  process.exit(1);
+});