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

(build error): (npx lerna run build --scope=aws-cdk-lib is failed with heap out of memory) #26415

Closed
bun913 opened this issue Jul 19, 2023 · 3 comments · Fixed by #26465
Closed
Labels
@aws-cdk/aws-appsync Related to AWS AppSync documentation This is a problem with documentation. effort/small Small work item – less than a day of effort p1

Comments

@bun913
Copy link
Contributor

bun913 commented Jul 19, 2023

Describe the issue

First, I executed the following command, referring to the following page, to set up the environment:

npx lerna run build --scope=aws-cdk-lib

https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md#building-aws-cdk-lib

However, I encountered the following error:

FATAL ERROR: Reached heap limit Allocation failed - JavaScript heap out of memory

I resolved this issue by increasing the available memory space by executing the following command:

export NODE_OPTIONS="--max-old-space-size=8192"

After that, I ran the command again:

npx lerna run build --scope=aws-cdk-lib

Proposal

I understand that this error is not caused by this repository but rather by the user's environment.

However, I believe that if the build fails, it could hinder the intentions of future contributors.

Would it be possible to add the following statement to CONTRIBUTING.md?

"If you encounter FATAL ERROR: Reached heap limit Allocation failed - JavaScript heap out of memory, please try running the command with increased memory by using export NODE_OPTIONS="--max-old-space-size=8192"."

I am also considering how far I should address this issue, so please let me know your opinion whether it is necessary or not if you think it is unnecessary.

Links

https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md#building-aws-cdk-lib

@bun913 bun913 added documentation This is a problem with documentation. needs-triage This issue or PR still needs to be triaged. labels Jul 19, 2023
@github-actions github-actions bot added the @aws-cdk/aws-appsync Related to AWS AppSync label Jul 19, 2023
@khushail
Copy link
Contributor

Hi @bun913 , thanks for reaching out. I agree with seeing this issue sometimes, so it would be helpful for the first time contributors to have this added to Contributing guide.

@khushail khushail added p2 effort/small Small work item – less than a day of effort p1 and removed needs-triage This issue or PR still needs to be triaged. p2 labels Jul 19, 2023
@bun913
Copy link
Contributor Author

bun913 commented Jul 20, 2023

@khushail
Thank you for your response. I will create a pull request for this issue today or tomorrow.

@mergify mergify bot closed this as completed in #26465 Jul 21, 2023
mergify bot pushed a commit that referenced this issue Jul 21, 2023
…or (#26465)

Closes #26415 

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
@github-actions
Copy link

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.

bmoffatt pushed a commit to bmoffatt/aws-cdk that referenced this issue Jul 29, 2023
…or (aws#26465)

Closes aws#26415 

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@aws-cdk/aws-appsync Related to AWS AppSync documentation This is a problem with documentation. effort/small Small work item – less than a day of effort p1
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants