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

logcli on OSX does not respect VPN DNS #1161

Closed
brokenjacobs opened this issue Oct 16, 2019 · 5 comments
Closed

logcli on OSX does not respect VPN DNS #1161

brokenjacobs opened this issue Oct 16, 2019 · 5 comments
Labels
keepalive An issue or PR that will be kept alive and never marked as stale.

Comments

@brokenjacobs
Copy link

Describe the bug
When connected over a VPN, on OSX, the logcli binary doesn't use the scutil resolvers configured by the vpn, instead it tries to parse /etc/resolv.conf and use the native go resolver. This is due to the way the go resolver system works on OSX.

To Reproduce
Steps to reproduce the behavior:
Connect with vpn.
use logcli
observe host not found error

Expected behavior
logcli connected to the appropriate server and working

Let's just cut to the resolution here, many go cli tools have suffered the same problem, including kubectl. The solution is to compile cgo support when building for darwin/osx. Here are the relevant kubectl issues:
kubernetes/kubernetes#23130
kubernetes/release#469
and some linked background material:
https://inconshreveable.com/04-30-2014/cross-compiling-golang-programs-with-native-libraries/

@stale
Copy link

stale bot commented Nov 15, 2019

This issue has been automatically marked as stale because it has not had any activity in the past 30 days. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale A stale issue or PR that will automatically be closed. label Nov 15, 2019
@stale stale bot closed this as completed Nov 22, 2019
@rfratto rfratto added the keepalive An issue or PR that will be kept alive and never marked as stale. label Jan 15, 2020
@rfratto rfratto reopened this Jan 15, 2020
@stale stale bot removed the stale A stale issue or PR that will automatically be closed. label Jan 15, 2020
@rfratto
Copy link
Member

rfratto commented Jan 15, 2020

Reopening since the PR to fix is also marked keepalive.

@mattmendick
Copy link
Contributor

Does the dns-heaven tool work to workaround this? Not a good solution for sure, but something that might help ease the pain.

@brokenjacobs
Copy link
Author

yes

@slim-bean
Copy link
Collaborator

I'm going to close this issue (and the associated PR) with the suggested dns-heaven workaround above. I'm not sure if this is the best solution but we haven't really heard other complaints about this and I don't want to leave it idle forever.

Please feel free to add comments or thumbs up this issue if the workaround doesn't work for you or you have other ideas, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
keepalive An issue or PR that will be kept alive and never marked as stale.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants