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 largest two RAM consumers, nimbus_beacon_node and all_tests, by default separately #4513

Merged
merged 1 commit into from
Jan 16, 2023

Conversation

tersec
Copy link
Contributor

@tersec tersec commented Jan 15, 2023

Hopefully mitigate the Windows OOMs; followup to #4495 #4502 #4504

$ make test
Building: install/usr/lib/libbacktracenim.a
Building: build/generate_makefile
Build completed successfully: build/generate_makefile
Building: build/consensus_spec_tests_mainnet
Building: build/consensus_spec_tests_minimal
Building: build/state_sim
Building: build/block_sim
... only later ...
Build completed successfully: build/state_sim
Build completed successfully: build/block_sim
Build completed successfully: build/consensus_spec_tests_minimal
Build completed successfully: build/consensus_spec_tests_mainnet
Building: build/all_tests

and similar for nbc

@etan-status
Copy link
Contributor

Oh, cool. Hope that finally makes Windows stable again. Appreciate all the effort, especially given the day/time

@github-actions
Copy link

Unit Test Results

         9 files  ±0    1 050 suites  ±0   33m 54s ⏱️ +20s
  3 512 tests ±0    3 275 ✔️ ±0  237 💤 ±0  0 ±0 
15 269 runs  ±0  15 004 ✔️ ±0  265 💤 ±0  0 ±0 

Results for commit f74b825. ± Comparison against base commit 7ad0d3e.

@tersec tersec enabled auto-merge (squash) January 16, 2023 10:33
@tersec tersec merged commit 87a34bf into unstable Jan 16, 2023
@tersec tersec deleted the fhy branch January 16, 2023 10:34
@tersec tersec mentioned this pull request Sep 12, 2023
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.

3 participants