Skip to content
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

Raise maximum Java heap used in Daily native image generation #1529

Merged

Conversation

michalvavrik
Copy link
Member

Summary

I thought the best way to mitigate Daily native image workflow running out of memory was to limit maximum Java heap used during the native image generation, but you can see that it didn't really help and I'm thinking GC just needs more. In both cases, we saw that we only allow GC small part of system memory. So either something else is using the memory (in which case we need to find it, like purge Docker images), or we simply don't use all that is at our disposal. The minimum memory suggested by Quarkus guide is 4 GB, so we can't go any lower.

See #1523

2023-11-19T03:40:09.2040280Z  - 3.56GB of memory (22.8% of 15.61GB system memory, set via '-Xmx4g')
2023-11-19T03:40:09.2040567Z  - 4 thread(s) (100.0% of 4 available processor(s), determined at start)
2023-11-19T03:40:09.2040833Z Terminating due to java.lang.OutOfMemoryError: GC overhead limit exceeded

Either this will work, or we need to clean resources, or build less of extensions. As daily build is already failing over this anyway, there is no risk at trying it.

Please select the relevant options.

  • Bug fix (non-breaking change which fixes an issue)
  • Dependency update
  • Refactoring
  • Backport
  • New scenario (non-breaking change which adds functionality)
  • This change requires a documentation update
  • This change requires execution against OCP (use run tests phrase in comment)

Checklist:

  • Methods and classes used in PR scenarios are meaningful
  • Commits are well encapsulated and follow the best practices

@gtroitsk
Copy link
Member

Let's try if this will work

@gtroitsk gtroitsk merged commit 8e6e93c into quarkus-qe:main Nov 19, 2023
1 check passed
@michalvavrik michalvavrik deleted the feature/give-native-daily-ci-more-xmx branch November 19, 2023 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants