Skip to content

fix(core): use next available port when the port for nx graph is in use #31365

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

FrozenPandaz
Copy link
Collaborator

Current Behavior

Command would fail silently with no error message

Expected Behavior

Shows clear error message: "Port 4211 is already in use. Please use a different
port with --port option or stop the process using port 4211."

Related Issue(s)

Fixes #30915

@FrozenPandaz FrozenPandaz requested a review from a team as a code owner May 28, 2025 01:27
@FrozenPandaz FrozenPandaz requested a review from Cammisuli May 28, 2025 01:27
Copy link

vercel bot commented May 28, 2025

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

1 Skipped Deployment
Name Status Preview Updated (UTC)
nx-dev ⬜️ Ignored (Inspect) Visit Preview May 28, 2025 9:50pm

Copy link

nx-cloud bot commented May 28, 2025

View your CI Pipeline Execution ↗ for commit 0e74b14.

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 2m 48s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 19s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 6s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded <1s View ↗
nx documentation ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2025-05-28 21:55:02 UTC

@FrozenPandaz FrozenPandaz force-pushed the fix-nx-graph-port-conflict branch 3 times, most recently from 61b59ef to c912529 Compare May 28, 2025 19:57
@FrozenPandaz FrozenPandaz force-pushed the fix-nx-graph-port-conflict branch from c912529 to 827dc85 Compare May 28, 2025 21:34
@FrozenPandaz FrozenPandaz force-pushed the fix-nx-graph-port-conflict branch from 827dc85 to 0e74b14 Compare May 28, 2025 21:47
@FrozenPandaz FrozenPandaz changed the title fix(core): throw an error when the port for nx graph is in use fix(core): use next available port when the port for nx graph is in use May 28, 2025
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.

Nx graph fails silently if port already taken
1 participant