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

Support for Ubuntu 17+ w/ systemd-resolved (.localhost only) #233

Merged
merged 2 commits into from Jan 24, 2020

Conversation

swrobel
Copy link
Collaborator

@swrobel swrobel commented Sep 11, 2019

Fixes #219


def resolver_file
using_systemd_resolved? ? nil : super
end
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If I understand this correctly, we don't need socat when systemd-resolvd is being used? So why do we even install socat.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be nice to read list of packages from same install_packages method here and for printing to user rather than duplicating the information in two places. But that duplication has always been there.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gnufied as far as I'm aware, socat performs port proxying (eg: domain.localhost:80 -> 127.0.0.1:9000). I tried on Ubuntu 18 without it and it required accessing domain.localhost:9000 but I'm open to having someone more experienced test

@gnufied
Copy link
Contributor

gnufied commented Sep 26, 2019

My own usage of invoker these days does not involve using it as a DNS resolver, so I would like to test this bit more before merging. Sorry about delay.

@swrobel
Copy link
Collaborator Author

swrobel commented Oct 30, 2019

@gnufied any luck w/ your testing?

@swrobel
Copy link
Collaborator Author

swrobel commented Dec 2, 2019

@gnufied ping!

@swrobel
Copy link
Collaborator Author

swrobel commented Jan 6, 2020

@gnufied happy new year 🎉 Any chance you've had a chance to test?

@swrobel swrobel changed the title Support for Ubuntu 17+ w/ systemd-resolved (.localhost domain only) Support for Ubuntu 17+ w/ systemd-resolved (.localhost domain on… Jan 24, 2020
@swrobel swrobel merged commit d3002f9 into codemancers:master Jan 24, 2020
@swrobel swrobel deleted the ubuntu-18-setup branch January 24, 2020 15:57
@swrobel swrobel changed the title Support for Ubuntu 17+ w/ systemd-resolved (.localhost domain on… Support for Ubuntu 17+ w/ systemd-resolved (.localhost only) Mar 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Invoker with Ubuntu 18.04
2 participants