Skip to content

Commit

Permalink
deny dynamic bootp clients if failover is set (#126)
Browse files Browse the repository at this point in the history
* deny dynamic bootp clients if failover is set

According to dhcpd.conf man page:
Dynamic BOOTP leases are not compatible with failover, and, as such,
you need to disallow BOOTP in pools that you are using failover for.

* rubocop: fix implicit expect
  • Loading branch information
aquister authored and jyaworski committed Nov 8, 2016
1 parent 6e44329 commit e800e47
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 1 deletion.
2 changes: 1 addition & 1 deletion spec/defines/pool6_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -18,5 +18,5 @@
}
end

it { should contain_concat__fragment("dhcp_pool_#{title}") }
it { is_expected.to contain_concat__fragment("dhcp_pool_#{title}") }
end
1 change: 1 addition & 0 deletions templates/dhcpd.pool.erb
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ subnet <%= @network %> netmask <%= @mask %> {
{
<% if @failover != '' -%>
failover peer "<%= @failover %>";
deny dynamic bootp clients;
<% end -%>
<% if @ignore_unknown == true -%>
ignore unknown-clients ;
Expand Down
1 change: 1 addition & 0 deletions templates/dhcpd.pool6.erb
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ subnet6 <%= @network %>/<%= @prefix %> {
{
<% if @failover != '' -%>
failover peer "<%= @failover %>";
deny dynamic bootp clients;
<% end -%>
<% if @ignore_unknown == true -%>
ignore unknown-clients ;
Expand Down

0 comments on commit e800e47

Please sign in to comment.