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

Separate Listening and Discovery Ports in Node Configuration #2308

Open
ItayPodhajcer opened this issue May 20, 2021 · 3 comments
Open

Separate Listening and Discovery Ports in Node Configuration #2308

ItayPodhajcer opened this issue May 20, 2021 · 3 comments
Labels
dev experience The build system, things that enable easier development etc. devops DevOps related task docker good first issue Good for newcomers icebox items that need more consideration, time, or can wait

Comments

@ItayPodhajcer
Copy link

Description

According to the documentation on P2P Port configuration, the --p2p-port options/BESU_P2P_PORT environment variable/p2p-port configuration file option can be used to configure the listening port for the node, which is used for both the TCP listening port and the UDP discovery port.

Strangely enough, there are sections in the documentation that describe a valid Enode URL that allows setting a different discovery port using a discport query parameter.

This brings the question of if those Enode URLs are supported, how can one configure a dedicated port for discovery, which has a different number than the listening port.

Acceptance Criteria

Allow configuring the listening port and discovery port of the node separatly,

Additional Information

It might seem like a small issue, but in scenarios that require exposing both ports, it creates limitations such as the cloud services that can be used to run nodes (at the time of opening this issue, both Azure ACI and AWS ECS do not support opening the same port number for both TCP and UDP at the same time).

@NicolasMassart
Copy link
Contributor

Hi @ItayPodhajcer,
Having two different ports for listening and advertising is not available in the base CLI use case, but if you use Besu with Kubernetes for instance, you can do it by setting services multiple port mappings and using the Besu Kubernetes NAT option.

Could you tell me if your use case includes using Kubernetes?

@ItayPodhajcer
Copy link
Author

@NicolasMassart Kubernetes is such an overkill in so many cases, especially when ACI/ECS are enough to get the job done.
Any plans to update the CLI in the near future?

@sajz sajz added the good first issue Good for newcomers label Jun 1, 2021
@vmichalik vmichalik added the dev experience The build system, things that enable easier development etc. label Sep 20, 2021
@TrishamBP
Copy link

@NicolasMassart How can we tackle this issue and make changes in our code? I have also been facing a lot of trouble getting my ports configured.

@non-fungible-nelson non-fungible-nelson added the icebox items that need more consideration, time, or can wait label May 24, 2022
@iamhsk iamhsk added the TeamChupa GH issues worked on by Chupacabara Team label Jul 14, 2022
@macfarla macfarla added devops DevOps related task and removed TeamChupa GH issues worked on by Chupacabara Team labels Jul 22, 2024
@jflo jflo added the docker label Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev experience The build system, things that enable easier development etc. devops DevOps related task docker good first issue Good for newcomers icebox items that need more consideration, time, or can wait
Projects
None yet
Development

No branches or pull requests

9 participants