nrwl / nx

Smart Monorepos · Fast CI
https://nx.dev
MIT License
23.75k stars 2.37k forks source link

chore(repo): disable parallelism for nx-dev:build-base and separate it from `test` tasks #28887

Closed FrozenPandaz closed 1 week ago

FrozenPandaz commented 1 week ago

Current Behavior

Running nx-dev:build-base takes a lot of memory and particularly if it is run after test tasks, it will cause agents to run out of memory.

Expected Behavior

I don't have a fix for the task taking up a lot of memory... but I do have a fix which will alleviate the issue. I've split up test tasks and nx-dev:build-base to run on different agents.

We should still look into how to make this task not take up so much memory. What I've learned is that prerendering pages for nx-dev has a memory leak so we should investigate what during prerendering a page is not cleaned up properly.

Related Issue(s)

Fixes #

vercel[bot] commented 1 week ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
nx-dev ✅ Ready (Inspect) Visit Preview Nov 13, 2024 2:26pm
nx-cloud[bot] commented 1 week ago

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 1b9e486c6f6f16f0d665f3a527f02036f324de0a. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 4 targets - [`nx affected --targets=lint,test,build,e2e,e2e-ci --base=dbaeb861023f2cd771ab3241ba9da059309e1fb2 --head=1b9e486c6f6f16f0d665f3a527f02036f324de0a --parallel=3`](https://staging.nx.app/runs/F9dPi1YSxs?utm_source=pull-request&utm_medium=comment) - [`nx run-many -t check-imports check-commit check-lock-files check-codeowners documentation --parallel=1 --no-dte`](https://staging.nx.app/runs/bmYP8LOkbP?utm_source=pull-request&utm_medium=comment) - [`nx-cloud record -- nx format:check --base=dbaeb861023f2cd771ab3241ba9da059309e1fb2 --head=1b9e486c6f6f16f0d665f3a527f02036f324de0a`](https://staging.nx.app/runs/dJ00UTcafh?utm_source=pull-request&utm_medium=comment) - [`nx documentation --no-dte`](https://staging.nx.app/runs/q6vSCzUYy2?utm_source=pull-request&utm_medium=comment)

Sent with 💌 from NxCloud.

github-actions[bot] commented 2 days ago

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.