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

hardcode proxy.replicated.com and replicated.app static IPs in troubleshoot output #1203

Merged

Conversation

laverya
Copy link
Member

@laverya laverya commented Sep 19, 2024

What this PR does / why we need it:

This simplifies client debugging by specifying the static IP addresses of proxy.replicated.com and replicated.app when connectivity checks to those domains fail.

We may want to consider adding these as compile time variables, but they really shouldn't be changing on our end either.

Which issue(s) this PR fixes:

Does this PR require a test?

Does this PR require a release note?


Does this PR require documentation?

sgalsaleh
sgalsaleh previously approved these changes Sep 19, 2024
@laverya laverya enabled auto-merge (squash) September 19, 2024 16:06
Copy link

github-actions bot commented Sep 19, 2024

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-2654466" -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-2654466?airgap=true" -H "Authorization: $EC_SMOKE_TEST_LICENSE_ID" -o embedded-cluster-smoke-test-staging-app-ci.tgz

Happy debugging!

@laverya laverya merged commit 1ab75c3 into main Sep 19, 2024
56 checks passed
@laverya laverya deleted the laverya/sc-111874/update-connectivity-preflights-to-mention branch September 19, 2024 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants