Add localhost as a resolver

[grosser]

@createdbypete

/cc @tmcinerney

[tmcinerney]

👍

[grosser]

@wfarr mergable ?

[grosser]

pong, merge ?

[dgoodlad]

👎

If we're changing this default, it's going to be contentious. Let's start with making it a configurable option (via hiera and a parameterized class), then the default can be changed independently.

[grosser]

What is bad about adding another resolve ?

Now using this fix ...

  # resolve localhost + localhost.com
  # https://github.com/boxen/puppet-dnsmasq/pull/9
  File <| title == "${dnsmasq::config::configdir}/dnsmasq.conf" |> {
    source => undef,
    content => inline_template("<%=
      root = scope.lookupvar('boxen::config::home') + '/repo'
      File.read(root + '/shared/dnsmasq/files/dnsmasq.conf') + File.read(root + '/modules/ours/files/dnsmasq.conf')
    %>")
  }

[dgoodlad]

@grosser taking over that zone completely and arbitrarily is not something I want to do without making it configurable.

[grosser]

now it's configurable, but still cannot add more, would this work ?

dnsmas::add: "address=/localhost.com/127.0.0.1\naddress=/localhost/127.0.0.1"