diff options
| author | Jakub Hrozek <jhrozek@redhat.com> | 2010-11-11 19:27:27 +0100 |
|---|---|---|
| committer | Simo Sorce <ssorce@redhat.com> | 2010-12-02 16:46:02 -0500 |
| commit | 88188cbb2050d4488df443a3f44454dd1867941f (patch) | |
| tree | 146d5bfd5af651bfb68cffc25a535a4d850515d2 /ipaserver | |
| parent | a41e69fba360d7cfc83e8592528562e06573533b (diff) | |
| download | freeipa-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')
| -rw-r--r-- | ipaserver/install/bindinstance.py | 12 |
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) |