Skip to content

Commit 4623fb5

Browse files
committed
Autogenerate reverse zones
1 parent 6e8652d commit 4623fb5

File tree

3 files changed

+52
-0
lines changed

3 files changed

+52
-0
lines changed

bind/config.sls

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
{% from "bind/map.jinja" import map with context %}
2+
{% from "bind/reverse_zone.jinja" import generate_reverse %}
23
34
include:
45
- bind
@@ -175,6 +176,9 @@ bind_rndc_client_config:
175176
{% for zone, zone_data in view_data.get('configured_zones', {}).items() -%}
176177
{%- set file = salt['pillar.get']("bind:available_zones:" + zone + ":file", false) %}
177178
{%- set zone_records = salt['pillar.get']('bind:available_zones:' + zone + ':records', {}) %}
179+
{%- if salt['pillar.get']('bind:available_zones:' + zone + ':generate_reverse') %}
180+
{%- do generate_reverse(zone_records, salt['pillar.get']('bind:available_zones:' + zone + ':generate_reverse:net'), salt['pillar.get']('bind:available_zones:' + zone + ':generate_reverse:for_zones'), salt['pillar.get']('bind:available_zones', {})) %}
181+
{%- endif %}
178182
{# If we define RRs in pillar, we use the internal template to generate the zone file
179183
otherwise, we fallback to the old behaviour and use the declared file
180184
#}

bind/reverse_zone.jinja

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
{%- macro generate_reverse(zone_records, net, for_zones, available_zones) %}
2+
{%- set res = {} %}
3+
{%- set net = salt['network.calc_net'](net) %}
4+
{% set for_zones = [ for_zones ] if for_zones is string else for_zones %}
5+
{%- for zone in for_zones %}
6+
{%- set A_records = available_zones.get(zone, {}).get('records', {}).get('A', {}) %}
7+
{%- for name, addr_list in A_records|dictsort %}
8+
{%- set addr_list = [ addr_list ] if addr_list is string or addr_list is number else addr_list %}
9+
{%- for addr in addr_list %}
10+
{%- if salt['network.ip_in_subnet'](addr, net) %}
11+
{%- if name.endswith('.') %}
12+
{%- set full_name = name %}
13+
{%- else %}
14+
{%- set full_name = name + '.' + zone + '.' %}
15+
{%- endif %}
16+
{%- set rev_ip = '.'.join(addr.split('.')[::-1]) %}
17+
{%- do res.update({rev_ip + '.in-addr.arpa.': full_name}) %}
18+
{%- endif %}
19+
{%- endfor %}
20+
{%- endfor %}
21+
{%- endfor %}
22+
{%- do zone_records.update({'PTR': res }) %}
23+
{%- endmacro %}

pillar.example

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -302,6 +302,31 @@ bind:
302302
- '"v=spf1 mx a ip4:1.2.3.4 ~all"'
303303
_dmarc: '"v=DMARC1; p=quarantine; rua=mailto:[email protected]; fo=1:d:s; adkim=r; aspf=r; pct=100; ri=86400"'
304304

305+
306+
307+
308+
3.2.1.in-addr.arpa: # auto-generated reverse zone
309+
file: example.com.rev.txt
310+
soa: # Declare the SOA RRs for the zone
311+
ns: ns1.example.com # Required
312+
contact: hostmaster.example.com # Required
313+
serial: auto # autoupdate serial on each change
314+
class: IN # Optional. Default: IN
315+
refresh: 8600 # Optional. Default: 12h
316+
retry: 900 # Optional. Default: 15m
317+
expiry: 86000 # Optional. Default: 2w
318+
nxdomain: 500 # Optional. Default: 1m
319+
ttl: 8600 # Optional. Not set by default
320+
records: # Records for the zone, grouped by type
321+
NS:
322+
'@':
323+
ns1.example.com.
324+
generate_reverse: # take all A records from example.com that are in 1.2.3.0/24 subnet
325+
net: 1.2.3.0/24 # and generate reverse records for them
326+
for_zones:
327+
- example.com # example.com is a zone defined in pillar, see above
328+
329+
305330
### Externally defined Zones ###
306331
bind:
307332
available_zones:

0 commit comments

Comments
 (0)