Skip to content

Allow Zone file to be created from a template #129

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.

Sign up for GitHub

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

Open
WetHippie wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Allow Zone file to be created from a template #129

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
e05a038
Provide a content option for the zone file so that it can be template…
Jun 6, 2017
8629453
Function is empty() not is_empty()
Jun 6, 2017
3d6656f
Don't delete the file after finishing so we can debug the update call.
Jul 5, 2017
734763e
Try spamming debug instead.
Jul 5, 2017
b1dbb1b
Debug on not-success.
Jul 5, 2017
0fdf1bf
Try a different place for debugs.
Jul 5, 2017
18d80cb
Simplify.
Jul 5, 2017
7c141bc
Print out just before running nsupdate.
Jul 5, 2017
e11ff07
Comment out debug println.
Jul 5, 2017
ed0d676
Debug print again for remote apex handling.
Jul 5, 2017
c213c50
Disable debug again.
Jul 5, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions lib/puppet_bind/provider/nsupdate.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,8 @@ def update(&block)
yield file
file.write "send\n"
file.close
#puts File.read(file.path)

Copy link
Owner

@inkblot inkblot Oct 14, 2017

Choose a reason for hiding this comment

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

Could you get rid of this please?

if keyed?
nsupdate('-y', tsig_param, file.path)
elsif keyfile?
Expand Down
29 changes: 21 additions & 8 deletions manifests/zone.pp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@
$forwarders = '',
$forward = '',
$source = '',
$content = '',
$forwarders_port = 53,
) {
# where there is a zone, there is a server
Expand Down Expand Up @@ -102,14 +103,26 @@
}

if member(['init', 'managed'], $zone_file_mode) {
file { "${cachedir}/${name}/${zone_file}":
ensure => present,
owner => $bind_user,
group => $bind_group,
mode => '0644',
replace => ($zone_file_mode == 'managed'),
source => pick($source, 'puppet:///modules/bind/db.empty'),
audit => [ content ],
if ! empty($content) {
file { "${cachedir}/${name}/${zone_file}":
ensure => present,
owner => $bind_user,
group => $bind_group,
mode => '0644',
replace => ($zone_file_mode == 'managed'),
content => $content,
audit => [ content ],
Copy link
Owner

@inkblot inkblot Oct 14, 2017

Choose a reason for hiding this comment

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

You'll need to get rid of the audit parameters on this and the other file resource. They've been removed on master, and that's the source of the merge conflict.

}
} else {
file { "${cachedir}/${name}/${zone_file}":
ensure => present,
owner => $bind_user,
group => $bind_group,
mode => '0644',
replace => ($zone_file_mode == 'managed'),
source => pick($source, 'puppet:///modules/bind/db.empty'),
audit => [ content ],
}
}
}

Expand Down