-
Notifications
You must be signed in to change notification settings - Fork 655
Select node version failed on custom deployment #2088
Comments
Is the error random, or does it always happen? If always, can you share a test repo that we can use to repro the issue? Thanks! |
If you wish you can use the production repo (it's just a project for my bachelor thesis) https://github.com/chillyistkult/ydm. The deploy.sh is generated by the azure cli and only modified to run a The related Azure repository is http://ydm.azurewebsites.net/ |
I tried pushing your repo to a new site, and I did not see the error:
What's interesting is that your error has "D:\home\site\wwwroot\dist\iisnode.yml", with the |
I think it happens after you push the repo a second time after the first succesful deployment. Don't ask my why, I hoped you could tell me. Kudo sync is configured to only copy the /dist directory which is fine and as I mentioned it also worked for me the first time on a fresh app environment. Then some error kicks in (like this one or Thread aborted error) and I revert the app environment because it doesn't look like the problems are app related. So as I didn't changed anything in the generated deploy.sh (beside the grunt task) I am a little bit lost here. |
I think I sort of see what's going on:
So I'd say the root of the issue is that there is a bit of a clash between having |
What I don't understand is why is SelectNodeVersion.js trying to start my server.js before the build process is finished and the output /dist folder is copied to wwwroot? I mean even if I change the path in "start" to "node server" it will fail when it tries to boot up the script in So as far as I understand Azure/Kudo actually tries to run the script described in package.json "start" before AND after the deployment process? So I need a solution to either have the directory path of Can I ask you what you would do in such a situation? |
SelectNodeVersion does not try to run anything. It tries to parse out the start script so it can correctly generate iisnode.yml with But it may be that this logic needs to be adapted to cover scenarios like yours. Is this a correct summary of your scenario:
|
Yeah that's exactly how it should work. The dist folder contains a client and a server directory after the application is build in repo root (the So if I change |
Hi @chillyistkult @davidebbo , I've similaire error , if you can help please, here my log error Thank's |
@nwabdou85 after weeks of investigations, I moved my infrastructure to Amazon AWS. Azure is just not made to use (or abuse) as a build instance at the moment. As there are no meaningful integrations with third party continuous integration providers like circleci or travis right now I would suggest save your time on that, because even if you resolve this error I can ensure that there are plenty of other problems you will run into (like sporadic timeouts during build and copy tasks). |
@chillyistkult thank's |
So I have fought through endless bugs and problems to get to a point were the first deployment of our nodejs application works to see that it is broken again without even touching anything. No idea what is wrong this time, maybe you guys have an idea.
And the deploy.sh
The text was updated successfully, but these errors were encountered: