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

Workaround for Jenkins builds failing with "Java heap space" errors #464

Merged
merged 1 commit into from
Feb 12, 2024
Merged

Workaround for Jenkins builds failing with "Java heap space" errors #464

merged 1 commit into from
Feb 12, 2024

Conversation

iloveeclipse
Copy link
Member

Fixes #463

@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 12, 2024 •
edited
Loading

Test Results

    9 files  ±0      9 suites  ±0   34m 33s ⏱️ -16s
2 183 tests ±0  2 179 ✅ ±0   4 💤 ±0  0 ❌ ±0 
6 639 runs  ±0  6 628 ✅ ±0  11 💤 ±0  0 ❌ ±0 

Results for commit aab0971. ± Comparison against base commit 4327aa9.

♻️ This comment has been updated with latest results.

@iloveeclipse
Copy link
Member Author

@laeubi : no idea what is wrong, but maven seem to read JVM argument as maven option:

Error:  Error executing Maven.
org.apache.commons.cli.ParseException: Unrecognized maven.config file entries: [mx2g]

@mickaelistria
Copy link
Contributor

I think you mean Jenkins, not Gerrit.

@iloveeclipse iloveeclipse changed the title Workaround for gerrit builds failing with "Java heap space" errors Workaround for Jenkins builds failing with "Java heap space" errors Feb 12, 2024
@iloveeclipse
Copy link
Member Author

I think you mean Jenkins, not Gerrit.

Doesn't work either.

@akurtakov
Copy link
Member

"Please add https://github.com/eclipse-platform/eclipse.platform.releng.aggregator/blob/master/.mvn/jvm.config to this repository as well, it should help with that OOM errors as currently only 1GB of 4G is used by default." but you put it in maven.config. It's well documented and explained in https://maven.apache.org/configure.html .

@iloveeclipse
Copy link
Member Author

"Please add https://github.com/eclipse-platform/eclipse.platform.releng.aggregator/blob/master/.mvn/jvm.config to this repository as well, it should help with that OOM errors as currently only 1GB of 4G is used by default." but you put it in maven.config. It's well documented and explained in https://maven.apache.org/configure.html .

I'm allowed to make mistakes, OK?

@akurtakov
Copy link
Member

"Please add https://github.com/eclipse-platform/eclipse.platform.releng.aggregator/blob/master/.mvn/jvm.config to this repository as well, it should help with that OOM errors as currently only 1GB of 4G is used by default." but you put it in maven.config. It's well documented and explained in https://maven.apache.org/configure.html .

I'm allowed to make mistakes, OK?

Sure, that's why I point you to the detailed documentation for reference in the future.

@iloveeclipse iloveeclipse merged commit 8d19a8e into eclipse-equinox:master Feb 12, 2024
9 checks passed
HannesWell added a commit to HannesWell/p2 that referenced this pull request Oct 12, 2024
@HannesWell
[Build] Specify options for the build JVM only in Jenkins pipeline
361b345 
The options where introduced for all builds of the p2 repo via
eclipse-equinox#464

Having it enabled can cause problems in GitHub workflows that are
sensitive to the printout.
@HannesWell HannesWell mentioned this pull request Oct 12, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Jenkins builds fail with Java heap space
3 participants
@iloveeclipse @mickaelistria @akurtakov