Closed grosser closed 10 years ago
:+1:
@wfarr mergable ?
pong, merge ?
:-1:
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.
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')
%>")
}
@grosser taking over that zone completely and arbitrarily is not something I want to do without making it configurable.
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"
@createdbypete
/cc @tmcinerney