-
Notifications
You must be signed in to change notification settings - Fork 166
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
Memory issue on fedora in latest V8 (8.8) requirement #2527
Comments
From nodejs/node#36139 (comment):
From nodejs/node#36139 (comment):
|
I've added the build agenda label to this in case nobody gets around to looking at this. I know that the WG members from Red Hat are busy this week. @rvagg had some suggestions in nodejs/node#36139 (comment) to see if clearing things up on the existing hosts helps. Otherwise we might look at either adding 2GiB of swap to the Fedora hosts (if we have the disk space) or bumping the allocated memory. |
I'm only just seeing this so don't have too much intelligent to add (such as why it's failing) other than:
Originally posted by @rvagg in nodejs/node#36139 (comment) |
@richardlau is the failure only on Fedora because the machines were configured with less memory or is it something specific to Fedora? |
@mhdawson I haven't found anything yet to suggest a Fedora specific issue vs a simple memory issue. |
@richardlau thanks, in terms of:
Is that something you will have time to do on one of the machines? |
@mhdawson I'm not sure. I don't have much work time available in the remainder of this week outside of the scheduled Red Hat meetings. I could make time next week. |
I updated those two machines, cleared workspaces and rebooted. Here's a green run for you for that problematic PR: https://ci.nodejs.org/job/node-test-commit-linux/39601/ We've historically targeted ~2Gb ~2 core machines in CI, they should be our most common configuration. If it were a universal memory problem then I'd expect to see it in more places than just one type of machine. My guess is that it's a bug in the toolchain that's been resolved. There were a number of toolchain updates in the big batch of updates installed, including gcc and glibc. The biggest memory hog on the machine is 🤷 we'll keep an eye on these machines but for now it seems to be addressed. |
Nice work ❤️ @rvagg |
I'm reopening because everytime there's a V8 update that requires to recompile everything I have to run CI many times hoping it passes. It also happens with |
Well .. centos7-arm64-gcc8 is interesting because it's got plenty of memory. I think we're dealing with too much parallelism on that machine. For all of the arm64 machines we have We are also migrating our arm64 machines to new hardware and it'll be a good opportunity to fix all of this. I was hoping to do some nice containerised arm64 infra like we have for our As for fedora, I'm still at a bit of a loss. But we do need to upgrade, we're stuck on 30 and 32 but should be on 34 (probably keep 32 as our "last"). I don't know why they stand out, they're running on the same spec hardware as many of our other VMs. They have JOBS set to 2 so shouldn't be overdoing it. |
arm64 machines updated to JOBS=12, systems updated and rebooted, I think they should be good to go now Maybe someone from RedHat might volunteer to update our Fedora systems? I usually take the older ones, reimage them with the latest fedora, then change labels so that fedora-latest is the new ones and fedora-last-latest is the ones that were previously newest (32 in this case). If not, I might get to it sometime, there's also Alpine to do I think. |
@rvagg Yep need to get on with that, but other critical stuff has come up - next week hopefullly (I'm on vacation until Tuesday now). Superficially Sounds like we're pretty much on the same page in terms of what it makes sense to do though :-) |
I've started this now (starting with test-digitalocean-fedora30-x64-1). Reimaging was fairly painless but I ran into https://www.digitalocean.com/community/questions/fedora-33-how-to-persist-dns-settings-via-etc-resolv-conf (with Fedora 34) meaning our playbooks failed until I went onto the machine and fixed the DNS settings (as per https://www.digitalocean.com/community/questions/fedora-33-how-to-persist-dns-settings-via-etc-resolv-conf?answer=66950). |
Did we ever try increasing the swap space on these machines? (since it looks from the output earlier in thisk issue that they had none.) |
I don't think so |
We have not. Is it easily done via Ansible? I'm up for trying. |
I've added swap to the two Fedora 32 hosts
|
Might also try adding swap to the Debian 10 hosts, e.g. https://ci.nodejs.org/job/node-test-commit-linux/nodes=debian10-x64/43295/console
|
Have added swap to |
We ended up having an informal meeting and not streaming. In retrospect we probably should have streamed but at the start we were not sure we were going to discuss too much. |
I see I got the wrong issue for the last comment. |
Seems like adding swap has resolved the issue, but leaving this open until we have added the swap addition to our ansible scripts. EDIT we agreed to add to manual instructions for now and then close this issue. |
This issue is stale because it has been open many days with no activity. It will be closed soon unless the stale label is removed or a comment is made. |
Do you know if that's increased from before? It could very well be that this version of V8 has tipped the memory requirements for compilation such that the 2GiB is no longer enough and we need to add either more dedicated or swap to bring the Fedora hosts on par with the others (4GiB seems to be what other similar hosts are on, nodejs/node#36139 (comment)). Maybe open an issue over in nodejs/build?
Originally posted by @richardlau in nodejs/node#36139 (comment)
The text was updated successfully, but these errors were encountered: