feat: allow additional ddns-* options

`ddns-rev-domainname` is added at the top level of the configuration and
in the subnet section too.
`ddns-domainname` is added to the subnet section as well because it's
possible to do so in configuration (ie. supose that you want hosts in
 a given subnet to have a different domain, for instance iot.foo.com).

`ddns-hostname` is added for host section because actually quite handy
to fix broken names that some client are generating.

Author: ekacnet

dhcpd/defaults.yaml

@@ -35,6 +35,7 @@ dhcpd:
   ldap_tls_ciphers: ''
   ldap_tls_randfile: ''
   ddns_update_style: ''
+  ddns_rev_domainname: ''
   ddns_domainname: ''
   update_static_leases: false
   authoritative: false

dhcpd/files/dhcpd.conf

@@ -139,6 +139,10 @@ ddns-update-style {{ dhcpd.ddns_update_style }};
 #ddns-update-style none;
 {%- endif %}
 
+{%- if dhcpd.ddns_rev_domainname %}
+ddns-rev-domainname "{{ dhcpd.ddns_rev_domainname }}";
+{%- endif %}
+
 {%- if dhcpd.ddns_domainname %}
 ddns-domainname "{{ dhcpd.ddns_domainname }}";
 {%- endif %}

dhcpd/files/host.jinja

@@ -25,6 +25,9 @@ host {{ host }} {
   {%- if 'hardware' in config %}
   hardware {{ config.hardware }};
   {%- endif %}
+  {%- if 'ddns_hostname' in config %}
+  ddns-hostname "{{ config.ddns_hostname }}";
+  {%- endif %}
   {%- if 'fixed_address' in config %}
   fixed-address {{ config.fixed_address }};
   {%- endif %}

dhcpd/files/subnet.jinja

@@ -41,6 +41,12 @@ subnet {{ subnet }} netmask {{ config.netmask }} {
 {%- if 'domain_name' in config %}
   option domain-name "{{ config['domain_name'] }}";
 {%- endif %}
+{%- if 'ddns_rev_domainname' in config %}
+  ddns-rev-domainname "{{ config['ddns_rev_domainname'] }}";
+{%- endif %}
+{%- if 'ddns_domainname' in config %}
+  ddns-domainname "{{ config['ddns_domainname'] }}";
+{%- endif %}
 {%- if 'domain_search' in config %}
   option domain-search "{{ config['domain_search']|join('","') }}";
 {%- endif %}

test/integration/default/files/_mapdata/amazonlinux-1.yaml

@@ -23,6 +23,7 @@ values:
       code: 252
       type: string
   ddns_domainname: ''
+  ddns_rev_domainname: ''
   ddns_update_style: ''
   default_lease_time: 600
   deny: unknown-clients

test/integration/default/files/_mapdata/amazonlinux-2.yaml

@@ -23,6 +23,7 @@ values:
       code: 252
       type: string
   ddns_domainname: ''
+  ddns_rev_domainname: ''
   ddns_update_style: ''
   default_lease_time: 600
   deny: unknown-clients

test/integration/default/files/_mapdata/arch-base-latest.yaml

@@ -23,6 +23,7 @@ values:
       code: 252
       type: string
   ddns_domainname: ''
+  ddns_rev_domainname: ''
   ddns_update_style: ''
   default_lease_time: 600
   deny: unknown-clients

test/integration/default/files/_mapdata/centos-7.yaml

@@ -23,6 +23,7 @@ values:
       code: 252
       type: string
   ddns_domainname: ''
+  ddns_rev_domainname: ''
   ddns_update_style: ''
   default_lease_time: 600
   deny: unknown-clients

test/integration/default/files/_mapdata/centos-8.yaml

@@ -23,6 +23,7 @@ values:
       code: 252
       type: string
   ddns_domainname: ''
+  ddns_rev_domainname: ''
   ddns_update_style: ''
   default_lease_time: 600
   deny: unknown-clients

test/integration/default/files/_mapdata/debian-10.yaml

@@ -23,6 +23,7 @@ values:
       code: 252
       type: string
   ddns_domainname: ''
+  ddns_rev_domainname: ''
   ddns_update_style: ''
   default_lease_time: 600
   deny: unknown-clients