-
Notifications
You must be signed in to change notification settings - Fork 10.7k
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
npm installation fails with Node.js 18 or 19 #6740
Comments
Hey @nobuto-m, Thanks for the report we have supported node 18 for a while and our docker images have moved to it for the official releases more recently. It looks like something is going on when trying to install the SQLite package, what hardware are you installing n8n on? |
I initially suspected OOM when building the SQLite. But it's reliably reproducible with a clean VM with 2 CPU cores and 8GB memory with the AMD64 (x86_64) architecture. |
I suspect this is an issue with |
Yup, that makes sense as a culprit. I will try it later and please consider this report as incomplete as it might be only a snap environment issue. |
Okay it was nothing to do with n8n so I moved this to nodejs/snap#56 |
@nobuto-m thanks for the update. |
Describe the bug
npm installation fails with Node.js 18 or 19 although the official installation doc states "n8n requires Node.js 18 or above".
https://docs.n8n.io/hosting/installation/npm/
To Reproduce
Steps to reproduce the behavior:
sudo snap install node --classic --channel 18
sudo -H npm install n8n -g
[/root/.npm/_logs/2023-07-26T04_21_42_281Z-debug-0.log]
It's reproducible with the following combination:
But it succeeds with the following in the meantime:
Expected behavior
Installation completes without any error.
Environment (please complete the following information):
own
,main
andqueue
. Default ismain
]Additional context
Node.js >= 18 looks like a relatively new requirement by #6649. However in my environment, it seems only Node.js >= 20 works. So if that's the case, it would be nice to update the official doc and package.json to avoid a confusion.
The text was updated successfully, but these errors were encountered: