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

fix: increase e2e test memory #1023

Merged
merged 2 commits into from
Feb 21, 2025
Merged

Conversation

jsy1218
Copy link
Member

@jsy1218 jsy1218 commented Feb 21, 2025

routing-api beta test keeps failing with out of memory:

[Container] 2025/02/21 04:41:24.456215 Command did not exit successfully set NODE_OPTIONS=--max-old-space-size=4096 && npm run test:e2e exit status 137
[Container] 2025/02/21 04:41:24.604544 Phase complete: BUILD State: FAILED
[Container] 2025/02/21 04:41:24.604566 Phase context status code: COMMAND_EXECUTION_ERROR Message: Error while executing command: set NODE_OPTIONS=--max-old-space-size=4096 && npm run test:e2e. Reason: exit status 137

We have to double the memory again

Copy link
Member Author

jsy1218 commented Feb 21, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@jsy1218 jsy1218 marked this pull request as ready for review February 21, 2025 04:59
@graphite-app graphite-app bot requested review from xrsv, cgkol and a team February 21, 2025 05:01
Copy link

graphite-app bot commented Feb 21, 2025

Graphite Automations

"Request reviewers once CI passes on routing-api repo" took an action on this PR • (02/21/25)

4 reviewers were added and 1 assignee was added to this PR based on Siyu Jiang (See-You John)'s automation.

Copy link
Member Author

jsy1218 commented Feb 21, 2025

Merge activity

  • Feb 21, 9:29 AM EST: A user started a stack merge that includes this pull request via Graphite.
  • Feb 21, 9:29 AM EST: A user merged this pull request with Graphite.

@jsy1218 jsy1218 merged commit d9a84d6 into main Feb 21, 2025
8 checks passed
@jsy1218 jsy1218 deleted the siyujiang/increase-e2e-test-memory branch February 21, 2025 14:29
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

Successfully merging this pull request may close these issues.

2 participants