Skip to content

fix(ec): 7443 only required on loopback interface #2026

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Mar 25, 2025
Merged

fix(ec): 7443 only required on loopback interface #2026

merged 1 commit into from
Mar 25, 2025

Conversation

@emosbaugh
Copy link
Member

@emosbaugh emosbaugh commented Mar 25, 2025 •
edited
Loading

What this PR does / why we need it:

ethan@ethanm-ec-3:~$ sudo netstat -tulpn | grep 7443
tcp        0      0 127.0.0.1:7443          0.0.0.0:*               LISTEN      449738/envoy
tcp        0      0 127.0.0.1:7443          0.0.0.0:*               LISTEN      449738/envoy
tcp        0      0 127.0.0.1:7443          0.0.0.0:*               LISTEN      449738/envoy
tcp        0      0 127.0.0.1:7443          0.0.0.0:*               LISTEN      449738/envoy
tcp        0      0 127.0.0.1:7443          0.0.0.0:*               LISTEN      449738/envoy
tcp        0      0 127.0.0.1:7443          0.0.0.0:*               LISTEN      449738/envoy
tcp        0      0 127.0.0.1:7443          0.0.0.0:*               LISTEN      449738/envoy
tcp        0      0 127.0.0.1:7443          0.0.0.0:*               LISTEN      449738/envoy

Which issue(s) this PR fixes:

Does this PR require a test?

Does this PR require a release note?

Host preflights have been updated to check that port 7443 is available on the loopback interface of the host, rather than being available on all network interfaces.

Does this PR require documentation?

replicatedhq/replicated-docs#3132

@emosbaugh emosbaugh marked this pull request as ready for review March 25, 2025 19:58
@github-actions
Copy link

github-actions bot commented Mar 25, 2025

This PR has been released (on staging) and is available for download with a embedded-cluster-smoke-test-staging-app license ID.

Online Installer:

curl "https://staging.replicated.app/embedded/embedded-cluster-smoke-test-staging-app/ci/appver-dev-922545c" -H "Authorization: $EC_SMOKE_TEST_LICENSE_ID" -o embedded-cluster-smoke-test-staging-app-ci.tgz

Airgap Installer (may take a few minutes before the airgap bundle is built):

curl "https://staging.replicated.app/embedded/embedded-cluster-smoke-test-staging-app/ci-airgap/appver-dev-922545c?airgap=true" -H "Authorization: $EC_SMOKE_TEST_LICENSE_ID" -o embedded-cluster-smoke-test-staging-app-ci.tgz

Happy debugging!

@emosbaugh emosbaugh enabled auto-merge (squash) March 25, 2025 20:22
@emosbaugh emosbaugh merged commit e755626 into main Mar 25, 2025
66 checks passed
@emosbaugh emosbaugh deleted the emosbaugh/sc-121580/add-preflight-for-7443-is-available-on-loopback branch March 25, 2025 22:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@laverya laverya laverya approved these changes

Assignees

No one assigned

Labels

type::feature

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@emosbaugh @laverya