From 69f0f1e59cf37b9a15a73681b63ba6da514a4148 Mon Sep 17 00:00:00 2001
From: Thomas Woerner <twoerner@redhat.com>
Date: Wed, 3 Apr 2019 16:02:07 +0200
Subject: [PATCH] ipareplica_setup_ds: Cleanup of unused dns settings in
 options

The dns settings in options have not been used and also not provided to
the module in the tasks file. Therefore these settings shoul dbe removed.
---
 roles/ipareplica/library/ipareplica_setup_ds.py | 9 ---------
 roles/ipareplica/tasks/install.yml              | 2 --
 2 files changed, 11 deletions(-)

diff --git a/roles/ipareplica/library/ipareplica_setup_ds.py b/roles/ipareplica/library/ipareplica_setup_ds.py
index c7ac7663..b0dacaf3 100644
--- a/roles/ipareplica/library/ipareplica_setup_ds.py
+++ b/roles/ipareplica/library/ipareplica_setup_ds.py
@@ -228,15 +228,6 @@ def main():
     if options.subject_base is not None:
         options.subject_base = DN(options.subject_base)
     options.ca_subject = ansible_module.params.get('ca_subject')
-    ### dns ###
-    options.reverse_zones = ansible_module.params.get('reverse_zones')
-    options.no_reverse = ansible_module.params.get('no_reverse')
-    options.auto_reverse = ansible_module.params.get('auto_reverse')
-    options.forwarders = ansible_module.params.get('forwarders')
-    options.no_forwarders = ansible_module.params.get('no_forwarders')
-    options.auto_forwarders = ansible_module.params.get('auto_forwarders')
-    options.forward_policy = ansible_module.params.get('forward_policy')
-
     ### additional ###
     #options._host_name_overridden = ansible_module.params.get(
     #    '_hostname_overridden')
diff --git a/roles/ipareplica/tasks/install.yml b/roles/ipareplica/tasks/install.yml
index 33ee62e0..6fd29299 100644
--- a/roles/ipareplica/tasks/install.yml
+++ b/roles/ipareplica/tasks/install.yml
@@ -250,8 +250,6 @@
       dirsrv_cert_files: "{{ ipareplica_dirsrv_cert_files | default([]) }}"
       ### client ###
       force_join: "{{ ipaclient_force_join }}"
-      ### dns ###
-      forward_policy: "{{ ipareplica_forward_policy | default(omit) }}"
       ### ad trust ###
       netbios_name: "{{ ipareplica_netbios_name | default(omit) }}"
       rid_base: "{{ ipareplica_rid_base | default(omit) }}"
-- 
GitLab