FYI - Applying 11/3/2022 and 11/4/2022 Commits in XO from Sources

hoerup

When I (re)build on my 3 GB vm i use this before in order to keep nodejs at bay

export NODE_OPTIONS='--max-old-space-size=3072'

And then it runs smoothly

Danp

@hoerup Unfortunately, that doesn't help in my situation. I recently upgraded the VM to Ubuntu 22.10, so maybe that is contribution to the problem.

JamfoFL

@Danp I'm using Debian Bullseye... I am able to run the updates now, but still see that same "chunk size" error notice you initially reported. Still, it seems you have it even worse...

ronivay

I'm running daily installation from sources on multiple different OS's. All have the same specs: 2vCPU/4GB RAM. This has worked flawlessly for a long time. Recently (starting from 4th/5th Nov) i've started to see OOM errors almost daily during yarn build which then cause it to fail with following error:

Using polyfills: No polyfills were added, since the `useBuiltIns` option was not set.
[01:23:25] Finished 'copyAssets' after 36 s
[01:24:33] Finished 'buildScripts' after 1.73 min
[01:24:33] Finished 'build' after 1.73 min
✖ 1
error Command failed with exit code 1.

It isn't consistent, sometimes it's debian that fails, sometimes ubuntu, sometimes centos/almalinux and so on. Something has definitely changed in the build procedure that eats more RAM than it used to.

I’m fine with increasing the RAM if needed. Just wanted to point this out if there’s something out of the ordinary with latest changes.

Danp

@ronivay said in FYI - Applying 11/3/2022 and 11/4/2022 Commits in XO from Sources:

Something has definitely changed in the build procedure that eats more RAM than it used to.

Agreed. Maybe @julien-f can add some insight into what has changed and how to successfully build from sources.

julien-f

I believe this was due to the inclusion of XO Lite on the master branch.

I've limited the number of packages built concurrently: https://github.com/vatesfr/xen-orchestra/commit/08298d3284119ad855552af36a810a3a9a006759

Tell me if that helps

0

julien-f committed to vatesfr/xen-orchestra

feat: limit concurrency of root build script

Should fixes https://xcp-ng.org/forum/post/54567

JamfoFL

@julien-f I just ran a "yard build" this morning, and other than still seeing the chunk error message:

(!) Some chunks are larger than 500 KiB after minification. Consider:
- Using dynamic import() to code-split the application
- Use build.rollupOptions.output.manualChunks to improve chunking: https://rollupjs.org/guide/en/#outputmanualchunks
- Adjust chunk size limit for this warning via build.chunkSizeWarningLimit.

Everything else ran fine... no errors or OOM issues.

julien-f

@JamfoFL Great!

Yes, the warning is unrelated (ping @pdonias).

ronivay

@julien-f haven't seen OOM's after this so looks good, thank you.

EliSmith

Thanks for sharing