vagrant-landrush / landrush

A Vagrant plugin that provides a simple DNS server for Vagrant guests
MIT License
666 stars 78 forks source link

Issue #211 Making DNS host configuration a Vagrant host capabiltiy #218

Closed hferentschik closed 8 years ago

hferentschik commented 8 years ago

@budhrg @LalatenduMohanty @bexelbie - here is the pull request for automatic configuration of DNS on "known" Linux system. The overall change seems big, but that is mainly due to the fact that I did a quite big refactoring to make host DNS configuration a Vagrant host capability. Any feeback? Otherwise I'll go ahead and cut a 1.1.0.beta.2.

LalatenduMohanty commented 8 years ago

@hferentschik +1 for configuring DNS as per host capability.

LalatenduMohanty commented 8 years ago

@hferentschik Which Linux distribution you have used to test the PR.

LalatenduMohanty commented 8 years ago

Also I do not see any documentation with the PR? or did I miss it?

hferentschik commented 8 years ago

@LalatenduMohanty

+1 for configuring DNS as per host capability.

Yeah, makes the code more uniform and easy to add new host types

Which Linux distribution you have used to test the PR.

Debian, Ubuntu, CentOS

Also I do not see any documentation with the PR? or did I miss it?

I think you did. Check the changes to the README