-
Notifications
You must be signed in to change notification settings - Fork 2
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
feat: add host preflights for all needed ports #1205
base: main
Are you sure you want to change the base?
feat: add host preflights for all needed ports #1205
Conversation
checks if all needed ports are available on the server prior to start the installation.
This PR has been released (on staging) and is available for download with a embedded-cluster-smoke-test-staging-app license ID. Online Installer:
Airgap Installer (may take a few minutes before the airgap bundle is built):
Happy debugging! |
make sure preflights are failing if some ports are in use.
outcomes: | ||
- fail: | ||
when: "connection-refused" | ||
message: Port 30000/TCP is required, but the connection was refused. This may indicate that the connection is being redirected to somewhere else on your network (perhaps a proxy configuration). |
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.
I need to better understand this one to comment on the wording, but this can be fine for now.
message: Port 30000/TCP is required, but the connection was refused. This may indicate that the connection is being redirected to somewhere else on your network (perhaps a proxy configuration). | ||
- fail: | ||
when: "address-in-use" | ||
message: Port 30000/TCP is required, but another process is already listening on it. Relocate the conflicting process to continue. |
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.
When we merge Ethan's port relocation changes, we can update this:
message: Port 30000/TCP is required, but another process is already listening on it. Relocate the conflicting process to continue. | |
message: Port 30000/TCP is required, but another process is already listening on it. To proceed, relocate the conflicting process or specify an available port using --admin-console-port. |
message: Port 50000/TCP is required, but the connection was refused. This may indicate that the connection is being redirected to somewhere else on your network (perhaps a proxy configuration). | ||
- fail: | ||
when: "address-in-use" | ||
message: Port 50000/TCP is required, but another process is already listening on it. Relocate the conflicting process to continue. |
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.
When we merge Ethan's port relocation changes, we can update this:
message: Port 50000/TCP is required, but another process is already listening on it. Relocate the conflicting process to continue. | |
message: Port 50000/TCP is required, but another process is already listening on it. To proceed, relocate the conflicting process or specify an available port using --local-artifact-mirror-port. |
What this PR does / why we need it:
Checks if all needed ports are available on the server prior to start the installation.