Skip to content

Comments

feat: use makefile#20450

Open
charlielye wants to merge 8 commits intonextfrom
cl/all_in_makefile
Open

feat: use makefile#20450
charlielye wants to merge 8 commits intonextfrom
cl/all_in_makefile

Conversation

@charlielye
Copy link
Contributor

@charlielye charlielye commented Feb 12, 2026

Let's just commit to the makefile and deal with issues.

  • Uses process substitution in denoises redact. Think this fixes some edge case where redact lingers and prevents denoise exiting.
  • npm_deploy mutates packages in a tmp copy to prevent clobbering tests underfoot.
  • bootstrap builds are now all makefile driven. Get rid of -makefile commands.
  • Put job counts on parallel call in ci.sh as using nproc is too low. Weak machines can spin up many strong machines.
  • Adds NUMA awareness to benchmarks. If CPU's requested is < cores per NUMA node, we wait until we can get all the CPU's on a single node, else we fallback to old behaviour and just grab whatever cores we can. This is probably satisfactory as higher core count benches are generally a different longer running category of benches.

@charlielye charlielye enabled auto-merge February 13, 2026 16:27
@charlielye charlielye added this pull request to the merge queue Feb 22, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 22, 2026
@charlielye charlielye added this pull request to the merge queue Feb 22, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 22, 2026
@charlielye charlielye added this pull request to the merge queue Feb 22, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 22, 2026
@charlielye charlielye enabled auto-merge February 23, 2026 10:55
@charlielye charlielye added this pull request to the merge queue Feb 23, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 23, 2026
@charlielye charlielye added this pull request to the merge queue Feb 23, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 23, 2026
@charlielye charlielye enabled auto-merge February 23, 2026 16:12
Let's just commit to the makefile and deal with issues.

* Uses process substitution in denoises redact. Think this fixes some edge case where redact lingers and prevents denoise exiting.
* `npm_deploy` mutates packages in a tmp copy to prevent clobbering tests underfoot.
* bootstrap builds are now all makefile driven. Get rid of `-makefile` commands.
* Put job counts on parallel call in ci.sh as using nproc is too low. Weak machines can spin up many strong machines.
* Adds NUMA awareness to benchmarks. If CPU's requested is < cores per NUMA node, we wait until we can get all the CPU's on a single node, else we fallback to old behaviour and just grab whatever cores we can. This is probably satisfactory as higher core count benches are generally a different longer running category of benches.
@charlielye charlielye added this pull request to the merge queue Feb 23, 2026
github-merge-queue bot pushed a commit that referenced this pull request Feb 23, 2026
Let's just commit to the makefile and deal with issues.

* Uses process substitution in denoises redact. Think this fixes some
edge case where redact lingers and prevents denoise exiting.
* `npm_deploy` mutates packages in a tmp copy to prevent clobbering
tests underfoot.
* bootstrap builds are now all makefile driven. Get rid of `-makefile`
commands.
* Put job counts on parallel call in ci.sh as using nproc is too low.
Weak machines can spin up many strong machines.
* Adds NUMA awareness to benchmarks. If CPU's requested is < cores per
NUMA node, we wait until we can get all the CPU's on a single node, else
we fallback to old behaviour and just grab whatever cores we can. This
is probably satisfactory as higher core count benches are generally a
different longer running category of benches.
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 23, 2026
@AztecBot
Copy link
Collaborator

Flakey Tests

🤖 says: This CI run detected 1 tests that failed, but were tolerated due to a .test_patterns.yml entry.

\033FLAKED\033 (8;;http://ci.aztec-labs.com/6f07ebc0a832b690�6f07ebc0a832b6908;;�):  yarn-project/end-to-end/scripts/run_test.sh simple src/e2e_p2p/gossip_network.test.ts (217s) (code: 0) group:e2e-p2p-epoch-flakes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants