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

Increase the heap of the Gradle runner #41083

Closed
wants to merge 2 commits into from

Conversation

alpar-t
Copy link
Contributor

@alpar-t alpar-t commented Apr 10, 2019

The Gradle Daemon actually running the build does so with a 2GB heap.
This PT changes the heap configuration of the gradle process that talks
with the daemon to trigger the builds and relies the messages.

Relates to #41061

The Gradle Daemon actually running the build does so  with a 2GB heap.
This PT changes the heap configuration of the gradle process that talks
with the daemon to trigger the builds and relies the messages.

Relates to elastic#41061
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@rjernst
Copy link
Member

rjernst commented Apr 10, 2019

Aren't these files generated when we update the gradle version we use? How do we ensure the changes are propagated then?

@alpar-t
Copy link
Contributor Author

alpar-t commented Apr 10, 2019

@rjernst that's right we would have to come up with something to make sure this is not lost. That's why I'm not marking the issue closed yet, but would like to get some certainty that this is indeed the heap that runs out and the new values work better.

@alpar-t
Copy link
Contributor Author

alpar-t commented Apr 12, 2019

I'm fairly sure that OOM is coming from the grade cli not the daemon, unfortunately there's no way to keep this customization in the wrapper script to see if it helps, so I will move to configure this in CI instead

@alpar-t alpar-t closed this Apr 12, 2019
@alpar-t alpar-t deleted the increase-gradle-runner-heap branch May 6, 2019 12:52
@mark-vieira mark-vieira added the Team:Delivery Meta label for Delivery team label Nov 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants