-
Notifications
You must be signed in to change notification settings - Fork 326
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
JavaScript heap out of memory on PNPM project #10123
Comments
Analyzing the sibling repository https://github.com/nl-design-system/denhaag works a bit better only getting a "PNPM failed to resolve dependencies for path 'package.json': NoSuchElementException: Key type is missing in the map." Only 1315 of 1970 dependencies are picked up see nl-design-system-denhaag-orthw.zip
|
Tried analyzing another repository within the same project https://github.com/nl-design-system/rijkshuisstijl-community
Manually running pnpm list --json --recursive --depth Infinity --prod actually works and returns
|
On advice of @sschuberth I tried an older version of ORT to see of the rewrite by @fviernau of the ORT's npm/pnpm/yarn had any impact - first tried 43.0.0 same issue then switched to 30.0.0. Analyzing the github.com/nl-design-system/utrecht project with ORT 30.0.0 works albeit it take more 20 minutes to complete. nl-design-system-utrecht-ort-30.0.0-analyzer-result.yml.zip For comparison I ran Trivy which took 8 seconds to produce a cyclonedx SBOM - albeit with a lot less information in it.
|
I also reran https://github.com/nl-design-system/denhaag with ORT 30.0.0 and it found this time 1522 dependencies with a proper dependency tree unlike ORT 55. ORT CycloneDX search "type" : "library" results in 1515. hits. Also ran trivy using See also nl-design-system-denhaag-ort-30.zip for all the result files. @fviernau Looks like ORT 30 did a better job with PNPM and we have a gap in detected dependencies compared to Trivy. |
For completeness i also analyzed https://github.com/nl-design-system/rijkshuisstijl-community with ORT 30.0.0 and it just works compared to 55.2.0 issues mentioned earlier in this issue. See for the result files nl-design-system-rijkshuisstijl-community-ort-30.zip. Again Trivy show more dependencies 2309 vs ORT's 1652. |
Note that this issue now mixes various different root causes. For example, the third post (and anything that mentions "Invalid string length") is covered by #9405. |
Regarding the memory issues, I wonder whether they're cause by the JSON produced by @tsteenbe can you try out my |
Actually, I just realized it's JavaScript, not Java, that's running out of memory:
So it again looks like an upstream PNPM bug. There seem to be several related issues. |
So, I've tried to increase Node's memory via
No need to try that out anymore, it does not help. |
Describe the bug
Getting JavaScript heap out of memory when running analyzer over https://github.com/nl-design-system/utrecht
To Reproduce
Steps to reproduce the behavior:
Expected behavior
An analyzer-result.yml result file with no eeors
Environment
Output of the
ort requirements
command:The text was updated successfully, but these errors were encountered: