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

The processlogs script sometimes throws "Memory limit of 1430 MiB exceeded" #3770

Closed
trivikr opened this issue Jun 19, 2024 · 2 comments
Closed

Comments

@trivikr
Copy link
Member

trivikr commented Jun 19, 2024

Describe the issue

The processlogs script sometimes throws "Memory limit of 1430 MiB exceeded"

Steps to reproduce

View if you have access https://cloudlogging.app.goo.gl/J3n4QFpWFxwmjosc7

Observed behavior

The request failed because either the HTTP response was malformed or connection to the instance had an error. Additional troubleshooting documentation can be found at: https://cloud.google.com/run/docs/troubleshooting#malformed-response-or-connection-error
While handling this request, the container instance was found to be using too much memory and was terminated. This is likely to cause a new container instance to be used for the next request to this revision. If you see this message frequently, you may have a memory leak in your code or may need more memory. Consider creating a new revision with more memory.

Memory limit of 1430 MiB exceeded with 1501 MiB used. Consider increasing the memory limit, see https://cloud.google.com/run/docs/configuring/memory-limits

Expected behavior

No Memory limit error

Additional context

Noticed while reviewing logs in #3697 (comment)

@trivikr
Copy link
Member Author

trivikr commented Jun 19, 2024

It would be beneficial to revisit this after updating Node.js to 20.x in #3767

@trivikr
Copy link
Member Author

trivikr commented Jul 5, 2024

Closing as this error hasn't been thrown since 2024-06-06, and we've recently done bunch of updates in metrics scripts which helped fix many other issues https://github.com/nodejs/build/commits/main/ansible/roles/metrics

@trivikr trivikr closed this as completed Jul 5, 2024
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

No branches or pull requests

1 participant