summaryrefslogtreecommitdiffstats
path: root/ipaserver/install
diff options
context:
space:
mode:
authorJakub Hrozek <jhrozek@redhat.com>2010-11-11 19:27:27 +0100
committerSimo Sorce <ssorce@redhat.com>2010-12-02 16:46:02 -0500
commit88188cbb2050d4488df443a3f44454dd1867941f (patch)
tree146d5bfd5af651bfb68cffc25a535a4d850515d2 /ipaserver/install
parenta41e69fba360d7cfc83e8592528562e06573533b (diff)
downloadfreeipa-88188cbb2050d4488df443a3f44454dd1867941f.tar.gz
freeipa-88188cbb2050d4488df443a3f44454dd1867941f.tar.xz
freeipa-88188cbb2050d4488df443a3f44454dd1867941f.zip
Do not create reverse zone by default
Prompt for creation of reverse zone, with the default for unattended installations being False. https://fedorahosted.org/freeipa/ticket/418
Diffstat (limited to 'ipaserver/install')
-rw-r--r--ipaserver/install/bindinstance.py12
1 files changed, 10 insertions, 2 deletions
diff --git a/ipaserver/install/bindinstance.py b/ipaserver/install/bindinstance.py
index 0f8200874..a10b85464 100644
--- a/ipaserver/install/bindinstance.py
+++ b/ipaserver/install/bindinstance.py
@@ -58,6 +58,11 @@ def check_inst(unattended):
return True
+def create_reverse(unattended):
+ if unattended:
+ return False
+ return ipautil.user_input("Do you want to configure the reverse zone?", False)
+
def dns_container_exists(fqdn, realm):
"""
Test whether the dns container exists.
@@ -200,13 +205,14 @@ class BindInstance(service.Service):
self.realm = None
self.forwarders = None
self.sub_dict = None
+ self.create_reverse = False
if fstore:
self.fstore = fstore
else:
self.fstore = sysrestore.FileStore('/var/lib/ipa/sysrestore')
- def setup(self, fqdn, ip_address, realm_name, domain_name, forwarders, ntp, named_user="named", zonemgr=None):
+ def setup(self, fqdn, ip_address, realm_name, domain_name, forwarders, ntp, create_reverse, named_user="named", zonemgr=None):
self.named_user = named_user
self.fqdn = fqdn
self.ip_address = ip_address
@@ -216,6 +222,7 @@ class BindInstance(service.Service):
self.host = fqdn.split(".")[0]
self.suffix = util.realm_to_suffix(self.realm)
self.ntp = ntp
+ self.create_reverse = create_reverse
if zonemgr:
self.zonemgr = zonemgr.replace('@','.')
@@ -247,7 +254,8 @@ class BindInstance(service.Service):
if not dns_container_exists(self.fqdn, self.suffix):
self.step("adding DNS container", self.__setup_dns_container)
self.step("setting up our zone", self.__setup_zone)
- self.step("setting up reverse zone", self.__setup_reverse_zone)
+ if self.create_reverse:
+ self.step("setting up reverse zone", self.__setup_reverse_zone)
self.step("setting up kerberos principal", self.__setup_principal)
self.step("setting up named.conf", self.__setup_named_conf)