Skip to content

Conversation

@tjsturos
Copy link
Contributor

No description provided.

@tjsturos tjsturos force-pushed the feat/add-worker-port-checks branch from e9fb8e1 to 3368f61 Compare December 19, 2025 07:31
@blacks1ne
Copy link
Contributor

Could you please explain the challenge and the approach?

@tjsturos
Copy link
Contributor Author

tjsturos commented Dec 19, 2025 via email

@blacks1ne
Copy link
Contributor

blacks1ne commented Dec 20, 2025

Thanks for the clarifications.

  1. Frankly speaking, I have not spotted any issues with workers on my side. But I am running standalone nodes. Are you also running a standalone node or a cluster?
  2. As you still seem to be experimenting, I would mark this PR as a DRAFT, implying it is not ready to be merged yet.
  3. If this feature could benefit only some node, it could be worthy to make it configurable instead of forcing default behavior.
  4. Once you complete the experiments and finalize the code, I could share some suggestions. For example, you have // Log but don't fail - port was available, but are not actually logging anything :)

@tjsturos tjsturos marked this pull request as draft December 20, 2025 19:22
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