commit b7f2ab474356908be8f29b34a712696c5503f781
author Zack Williams <zdw@opennetworking.org> Mon Mar 15 22:26:41 2021 -0700
committer Zack Williams <zdw@opennetworking.org> Wed Mar 24 14:14:51 2021 -0700
tree ae04181f79c8088c1ceb3817d5db7e80d3e082b7
parent da1625e24935d0887dee592f1bf1a55d8ae984fd

[AETHER-1000]

Allow reverse DNS lookups to work

NOTE: requires change in syntax used to define a zone - dns_zones no
longer works, must use dns_forward_zones/dns_reverse_zones

Fix multiplatform support

Change-Id: Ibd416b2da8853bc0b25ddec1774ddf9a9e1bf898

molecule/default/converge.yml

diff --git a/molecule/default/converge.yml b/molecule/default/converge.yml
index e88b105..1b9266f 100644
--- a/molecule/default/converge.yml
+++ b/molecule/default/converge.yml
@@ -7,7 +7,7 @@
 - name: Converge
   hosts: all
   vars:
-    dns_zones:
+    dns_forward_zones:
       example.com:
         serial: 20201102
         ip_range: 10.0.0.1/24
@@ -26,12 +26,25 @@
         serial: 20201102
         ip_range: 10.0.10.1/24
         ns:
-          - gw2.example.com.
+          - gw2.example.org.
         a:
           gw2: 10.0.10.1
         cname: {}
         srv: {}
         txt: {}
+    dns_reverse_zones:
+      10.0.0.0/8:
+        serial: 20210315
+        ns:
+          - gw.example.com.
+          - gw2.example.org.
+        ptr:
+          10.0.0.1: gw.example.com.
+          10.0.0.2: host1.example.com.
+          10.0.0.3: host2.example.com.
+          10.0.0.4: printer1.example.com.
+          10.0.10.1: gw2.example.org.
+
   tasks:
     - name: "Include nsd"
       include_role:

molecule/default/verify.yml

diff --git a/molecule/default/verify.yml b/molecule/default/verify.yml
index 43cba44..596330b 100644
--- a/molecule/default/verify.yml
+++ b/molecule/default/verify.yml
@@ -21,12 +21,24 @@
 
     - name: Check for NS record
       command:  # noqa 301
-        cmd: "dig ns example.com @127.0.0.1"
+        cmd: "dig ns +short example.com @127.0.0.1"
       register: ns_dig
-      failed_when: "'example.com.\t\t3600\tIN\tNS\tgw.example.com.' not in ns_dig.stdout"
+      failed_when: "'gw.example.com.' not in ns_dig.stdout"
 
     - name: Check for CNAME record
       command:  # noqa 301
         cmd: "dig lpr.example.com @127.0.0.1"
       register: cn_dig
       failed_when: "'lpr.example.com.\t3600\tIN\tCNAME\tprinter.example.com.' not in cn_dig.stdout"
+
+    - name: Check for reverse IP lookup
+      command:  # noqa 301
+        cmd: "dig -x {{ item.key }} @127.0.0.1"
+      register: rip_dig
+      failed_when: "item.value not in rip_dig.stdout"
+      with_dict:
+        10.0.0.1: gw.example.com.
+        10.0.0.2: host1.example.com.
+        10.0.0.3: host2.example.com.
+        10.0.0.4: printer1.example.com.
+        10.0.10.1: gw2.example.org.