-
Notifications
You must be signed in to change notification settings - Fork 210
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Deployment fails #30
Comments
I am experiencing the same. |
@davidebbo any chance you could have a look at this? |
Try looking at the |
Other thing to try: after the failure, go to Portal under Deployment Source for the site, and look at the deployment log |
I am getting the same failure error attempting to deploy. When looking at the deploy logs in Publishing -> Deploy source it shows the failure reason as "REASON 🔧 Update to Node v4". The log is attached. |
Seems the errors all start with:
I wonder if something changed on npm that messes it up. I'm guessing that trying to run npm install manually in site\wwwroot from Kudu console would fail in the same way? |
I will do a bit more debugging in a little while, but it seems changing to http instead of https for the registry.npmjs.org fetches resolves the issue. |
@davidebbo: Any reason why Web Apps wouldn't be able to establish a connection to npm? I don't think there's anything we can do from the repo here, is there? |
I can't think of any. We need to isolate this further. e.g.
|
What if you directly try to install those same packages in Kudu Console? e.g. |
I think it has to do with exceeding some connection quota in the sandbox. I need to dig deeper. Are you all seeing this on Free/Shared sites? In theory, this should not happen in Basic/Standard sites. |
I attempted to set up the Ghost install on a Free-tier, but I can try it out on a higher tier later. |
Just tried to spin it up on a Basic tier, and that seemed to work correctly. |
Yes, here is the deal:
This appears to be a known issue with npm. e.g. read the more recent comments from npm/npm#7200, with people complaining about it (not on Azure). I don't know why this only started to affect Ghost now, but I'd guess the number of dependencies might have increased and put it over the edge :( |
Thanks @davidebbo - thats fixed it for me too. A brand new deploy to a basic plan worked, and scaling my free plan up to basic for the deploy and then back down to free also worked. |
Not sure what is going on, but I just did a Standard w/ Medium size worker and got this.. Sits on building for quite some time until it finally failed.
|
@cicorias That looks like a very separate issue, discussed on npm/registry#10. People are seeing this on EC2 and other places. It seems for that one upgrading npm is the answer. Also, I'm not sure those errors are actually harmful. Back to the |
@davidebbo - ok - seems that others have been able to get passed that issue with a different version of node - Just so you know this was failing direct from the "Deploy to Azure" button integration. But, once that failed via Kudu console an If it's still an issue i'll open another issue. |
Someone else reporting this here. @felixrieseberg any chance to upgrade so it uses newer npm? |
Ghost supports the Node LTS release (so we're stuck with the latest LTS for now), but I set the npm version manually to the latest available on Azure (3.8.6). @davidebbo: That too is a few versions behind the latest available (3.8.9 is the latest patch, 3.9.1 the latest minor). This is a total off-topic, but let me know if we should build you a little script that just shovels all available versions into the image! |
there are npm versions in
|
Yes, exactly, please notice how 3.8.6 is the last available one ;-) |
also, instead of updating or specifying the node version in the
|
I'd like to change the original Ghost files as little as possible - the only reason I'm locking down the Node version at all is to ensure that pre-gyp (on the free tier) is able to find binaries for sqlite3 (which usually takes a while after a Node release). This is basically a long flowchart of "if we could [...] then we wouldn't have to [...]" 🏄 |
One problem I found is that Kudu doesn't support the caret syntax (e.g. ^4.2.0), because it's using an ancient semver.js from days where the syntax didn't exist. I think that's at least partially the root of some issues. I'll look into fixing that. |
This should be solved by this PR: #34. You can try it before it gets merged on my fork: https://github.com/davidebbo-test/Ghost-Azure |
I just tried a couple of deploys, but both times it failed:
Are the any logs so I can try to help find a fix?
These are the screenshots from the Azure Portal:
The text was updated successfully, but these errors were encountered: