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

Adjust the JVM options to heal the Native build on CI #4623

Merged
merged 1 commit into from
Jun 17, 2024

Conversation

danicheg
Copy link
Member

It's been five years since these options were last touched. Too many things have become complicated since then. But let's see if this actually fixes the Native build.

@danicheg
Copy link
Member Author

Mentioning the original issue with the Native build:

[error] Test runner interrupted by fatal signal 11
[warn] Force close java.lang.RuntimeException: Process /home/runner/work/cats/cats/tests/native/target/scala-2.13/cats-tests-test finished with non-zero value 139 (0x8b)

@danicheg
Copy link
Member Author

One might speculate that solving infrastructural issues by simply increasing resources isn't wise. It's a fair point. On the other hand, we've had three consecutive 'green' CI runs with these tweaks (#1, #2, #3).

Comment on lines +2 to +3
-Xms2G
-Xmx8G
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice. GitHub recently upgraded their runners so we have 16GB memory (I think the old limit was 7GB). We can't give it all to the JVM because Native (and JS) run their test suites as a separate process.

https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners/about-github-hosted-runners#standard-github-hosted-runners-for-public-repositories

@danicheg danicheg merged commit dd89133 into typelevel:main Jun 17, 2024
16 checks passed
@danicheg danicheg deleted the increase-resources branch June 17, 2024 08:31
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.

3 participants