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

Internet connectivity check fails if ICMP traffic is restricted #3978

Closed
asmithdt opened this issue May 19, 2023 · 1 comment · Fixed by #4284
Closed

Internet connectivity check fails if ICMP traffic is restricted #3978

asmithdt opened this issue May 19, 2023 · 1 comment · Fixed by #4284
Labels
kind/bug Something isn't working

Comments

@asmithdt
Copy link

Summary

When trying to install on macOS I get a failure to connect to the internet, the check is done via a ping (ICMP) traffic which isn't allowed on my corp network. However http/https (which are the connections actually required) are fine. I can't see any other use of ping other than a test.

What Should Happen Instead?

The internet check should check for http(s) connectivity instead of using ping.

Reproduction Steps

  1. be on network that disallows ICMP traffic
  2. microk8s install

Introspection Report

N/A

Can you suggest a fix?

https://github.com/canonical/microk8s/blob/master/installer/vm_providers/_base_provider.py#L131

curl --fail --head https://snapcraft.io instead perhaps?

Are you interested in contributing with a fix?

Yes

@neoaggelos neoaggelos added the kind/bug Something isn't working label May 23, 2023
@neoaggelos
Copy link
Contributor

Hi @asmithdt

Thank you for raising this, this has also been discussed in the past, but I do agree that your proposal sounds quite sound.

We could even attempt both (try the ping, then fallback to curl if that does not work), and if any of the two works then proceed with the installation assuming that we have network connectivity.

Contirbutions are welcome! Otherwise we will work on this soon. Thanks for raising this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants