0

Has anyone found a way to force azure to gracefully fail a deploy if it cannot start the node server?

There was a bug in our node server scripts, and I was only able to find it by pulling dump logs and tracing through them. I am updating the script to better catch and log the error, but I am unable to find a good way for it to fail the deployment if it cannot start the server.

I have found many people that are getting their deployments to fail, but I want mind to fail and cannot seem to force it in a scenario in which I believe it should.

Here is snippet from server.js

catch (err) { console.error(err); process.exit(1); }

kwelch
  • 2,370
  • 22
  • 23
  • Why would you like to force the deployment? Do you want to add a step during the deployment to test your application? If your application fail in test script, you want fail the deployment? – Gary Liu Aug 16 '16 at 06:25
  • So tests are ran during deployment? This would be a good option. I want the deploy to fail because now the server does not start and it is unclear why. Our currently build/test phase is compiling our react app, the node server is simplistic and adding to the codebase during deploy. – kwelch Aug 16 '16 at 18:22

1 Answers1

0

You can try to leverage the Custom Deployment Script. Usually, we will leverage this to do some additionally steps during the deployment task, like execute gulp or Browserify scripts to build up the production application from source code scripts. And if test scripts leverage pure node.js scripts, they are should be in the same mechanism.

Also, leveraging the custom deployment script, you can control the deployment process.

For node.js application, you can follow https://github.com/projectkudu/kudu/wiki/Custom-Deployment-Script#a-first-lets-generate-a-deployment-script-for-the-nodejs-website to generate the required deployment scripts, and you can modify the scripts and put them into your root directory of your application, then deploy them together to Azure.

Any further concern, please feel free to let me know.

Gary Liu
  • 13,758
  • 1
  • 17
  • 32
  • I contemplated going that route using start script but I saw that as bad form. Calling test script make perfect sense. Thanks! – kwelch Aug 17 '16 at 03:06