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.

Sign up for GitHub

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

Jump to bottom

CI Failures due to OOM #4030

Closed
rcaudy opened this issue Jun 20, 2023 · 1 comment · Fixed by #4035
Closed

CI Failures due to OOM #4030

rcaudy opened this issue Jun 20, 2023 · 1 comment · Fixed by #4035
Assignees
@jmao-denver @devinrsmith
Labels
bug Something isn't working ci
Milestone
June 2023

Comments

@rcaudy
Copy link
Member

rcaudy commented Jun 20, 2023

We are routinely getting random-ish failures in CI caused by OutOfMemoryErrors. For example:
https://github.com/deephaven/deephaven-core/actions/runs/5319076205/jobs/9631170540

This seems to have begun with a recent change to tighten heap provisions to various test components. Perhaps we need to revisit the sizing, and/or change something about the tests themselveS?
@rcaudy rcaudy added bug Something isn't working ci labels Jun 20, 2023
@rcaudy rcaudy added this to the June 2023 milestone Jun 20, 2023
@devinrsmith
Copy link
Member

It might also be somewhat related to gradle (potentially) taking up more memory:

gradle/gradle#23215

@devinrsmith devinrsmith mentioned this issue Jun 21, 2023
Merged
devinrsmith added a commit that referenced this issue Jun 21, 2023
@devinrsmith
Improve heap usage configurations (#4035)
e9715de 
Add commentary on org.gradle.jvmargs and update based on gradle build scans.
Upgrade Integration python testing maxmem to 2G.
Rename testing start_jvm to start_jvm_for_tests to disambiguate between other start_jvm functions.

Fixes #4030
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jmao-denver jmao-denver

@devinrsmith devinrsmith

Labels
bug Something isn't working ci
Projects
None yet
Milestone
June 2023
Development

Successfully merging a pull request may close this issue.

Improve heap usage configurations devinrsmith/deephaven-core
3 participants
@jmao-denver @rcaudy @devinrsmith