-
Notifications
You must be signed in to change notification settings - Fork 31
Add EC port requirements #2680
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
Add EC port requirements #2680
Changes from all commits
0f118a0
02b5a59
d8ea08e
9e95818
fcf9028
f3575d0
79beb8f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,24 @@ | ||
| Embedded Cluster requires that the following ports are open and available: | ||
|
|
||
| * 2379/TCP * | ||
| * 2380/TCP | ||
| * 4789/UDP | ||
| * 6443/TCP | ||
| * 7443/TCP | ||
| * 9091/TCP | ||
| * 9099/TCP * | ||
| * 9443/TCP | ||
| * 10248/TCP * | ||
| * 10249/TCP | ||
| * 10250/TCP | ||
| * 10256/TCP | ||
| * 10257/TCP * | ||
| * 10259/TCP * | ||
| * 30000/TCP *** | ||
| * 50000/TCP * ** *** | ||
|
|
||
| * These ports are used only by processes running on the same node. Ensure that there are no other processes using them. It is not necessary to create firewall openings for these ports. | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ^ @chris-sanders could you confirm that this language about the ports that need to be listening on localhost makes sense? |
||
|
|
||
| ** Required for air gap installations only. | ||
|
|
||
| *** By default, the Admin Console and Local Artifact Mirror (LAM) run on ports 30000 and 50000, respectively. If these ports are occupied, you can select different ports during installation. For more information, see [Change the Admin Console and LAM Ports](/enterprise/installing-embedded#adm-lam-ports). | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ^ also noticed that we weren't calling out how these change be changed |
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's merge this first, but just thinking out loud, I wonder if a table view here could be easier to parse. Columns like port number, protocol (tcp vs udp), local or not, air gap/online/both, etc. This might be perfectly fine too, just a thought.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah that would be worth a shot!