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

Test: Increase --max-old-space-size option for 2.21 #196

Closed
wants to merge 4 commits into from
Closed

Test: Increase --max-old-space-size option for 2.21 #196

wants to merge 4 commits into from

Conversation

huonw
Copy link
Contributor

@huonw huonw commented Apr 27, 2024

Testing if #195 fixes the build of #193.

@huonw huonw added the automation:sync-docs An automatic sync of documentation from the pantsbuild/pants repo label Apr 27, 2024
This was referenced Apr 27, 2024
Merged
Merged
@huonw
Copy link
Contributor Author

huonw commented Apr 27, 2024

Seems to work...

@huonw huonw closed this Apr 27, 2024
@huonw huonw deleted the huonw/more-memory-test branch April 27, 2024 08:32
huonw added a commit that referenced this pull request Apr 28, 2024
@huonw
Increase --max-old-space-size option for 2.21 (#195)
d3b2f41 
It appears that we need more memory to build after branching for 2.21:
the update that splits it (#193) is failing due to running out of
memory.

This bumps to 12GiB, slightly less than the runner's RAM (16GB).

(#196 validates that this does actually fix it for 2.21.)
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
automation:sync-docs An automatic sync of documentation from the pantsbuild/pants repo
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@huonw @WorkerPants