diff options
author | Martin Kosek <mkosek@redhat.com> | 2013-10-09 16:13:19 +0200 |
---|---|---|
committer | Martin Kosek <mkosek@redhat.com> | 2013-10-11 09:47:59 +0200 |
commit | cb9331ebfcc33ad676b6d188156a6d56b2996a18 (patch) | |
tree | 9f07da1d1c6de28f4ffe65a784dea72e773c8adf | |
parent | 5e8aab8558874a9a826a1c470e806c75fb84eef2 (diff) | |
download | freeipa-cb9331ebfcc33ad676b6d188156a6d56b2996a18.tar.gz freeipa-cb9331ebfcc33ad676b6d188156a6d56b2996a18.tar.xz freeipa-cb9331ebfcc33ad676b6d188156a6d56b2996a18.zip |
Remove --no-serial-autoincrement
Deprecate this option and do not offer it in installation tools.
Without this option enabled, advanced DNS features like DNSSEC
would not work.
https://fedorahosted.org/freeipa/ticket/3962
-rw-r--r-- | install/share/bind.named.conf.template | 2 | ||||
-rwxr-xr-x | install/tools/ipa-dns-install | 6 | ||||
-rw-r--r-- | install/tools/ipa-server-install | 4 | ||||
-rw-r--r-- | install/tools/man/ipa-dns-install.1 | 3 | ||||
-rw-r--r-- | install/tools/man/ipa-server-install.1 | 3 | ||||
-rw-r--r-- | ipaserver/install/bindinstance.py | 7 |
6 files changed, 3 insertions, 22 deletions
diff --git a/install/share/bind.named.conf.template b/install/share/bind.named.conf.template index 5727a153..0984febb 100644 --- a/install/share/bind.named.conf.template +++ b/install/share/bind.named.conf.template @@ -45,5 +45,5 @@ dynamic-db "ipa" { arg "auth_method sasl"; arg "sasl_mech GSSAPI"; arg "sasl_user DNS/$FQDN"; - arg "serial_autoincrement $SERIAL_AUTOINCREMENT"; + arg "serial_autoincrement yes"; }; diff --git a/install/tools/ipa-dns-install b/install/tools/ipa-dns-install index 47acd8de..37a07f8e 100755 --- a/install/tools/ipa-dns-install +++ b/install/tools/ipa-dns-install @@ -52,9 +52,6 @@ def parse_options(): parser.add_option("--zonemgr", action="callback", callback=bindinstance.zonemgr_callback, type="string", help="DNS zone manager e-mail address. Defaults to hostmaster@DOMAIN") - parser.add_option("--no-serial-autoincrement", dest="serial_autoincrement", - default=True, action="store_false", - help="Do not enable SOA serial autoincrement") parser.add_option("-U", "--unattended", dest="unattended", action="store_true", default=False, help="unattended installation never prompts the user") @@ -209,8 +206,7 @@ def main(): print "" bind.setup(api.env.host, ip_address, api.env.realm, api.env.domain, - dns_forwarders, conf_ntp, reverse_zone, zonemgr=options.zonemgr, - serial_autoincrement=options.serial_autoincrement) + dns_forwarders, conf_ntp, reverse_zone, zonemgr=options.zonemgr) bind.create_instance() # Restart http instance to make sure that python-dns has the right resolver diff --git a/install/tools/ipa-server-install b/install/tools/ipa-server-install index 8077bc20..b871ef3f 100644 --- a/install/tools/ipa-server-install +++ b/install/tools/ipa-server-install @@ -212,9 +212,6 @@ def parse_options(): help="Do not use DNS for hostname lookup during installation") dns_group.add_option("--no-dns-sshfp", dest="create_sshfp", default=True, action="store_false", help="Do not automatically create DNS SSHFP records") - dns_group.add_option("--no-serial-autoincrement", dest="serial_autoincrement", - default=True, action="store_false", - help="Do not enable SOA serial autoincrement") parser.add_option_group(dns_group) uninstall_group = OptionGroup(parser, "uninstall options") @@ -1181,7 +1178,6 @@ def main(): bind = bindinstance.BindInstance(fstore, dm_password) bind.setup(host_name, ip_address, realm_name, domain_name, dns_forwarders, options.conf_ntp, reverse_zone, zonemgr=options.zonemgr, - serial_autoincrement=options.serial_autoincrement, ca_configured=setup_ca) if options.setup_dns: api.Backend.ldap2.connect(bind_dn=DN(('cn', 'Directory Manager')), bind_pw=dm_password) diff --git a/install/tools/man/ipa-dns-install.1 b/install/tools/man/ipa-dns-install.1 index 646601ee..bde30cad 100644 --- a/install/tools/man/ipa-dns-install.1 +++ b/install/tools/man/ipa-dns-install.1 @@ -49,9 +49,6 @@ Do not create new reverse DNS zone. If used on a replica and a reverse DNS zone \fB\-\-zonemgr\fR The e\-mail address of the DNS zone manager. Defaults to hostmaster@DOMAIN .TP -\fB\-\-no\-serial\-autoincrement\fR -Do not enable SOA serial autoincrement feature. SOA serial will have to be updated automatically or other DNS features like zone transfer od DNSSEC will not function properly. This feature requires persistent search zone update mechanism. -.TP \fB\-U\fR, \fB\-\-unattended\fR An unattended installation that will never prompt for user input .SH "EXIT STATUS" diff --git a/install/tools/man/ipa-server-install.1 b/install/tools/man/ipa-server-install.1 index a093078c..4adf1d03 100644 --- a/install/tools/man/ipa-server-install.1 +++ b/install/tools/man/ipa-server-install.1 @@ -151,9 +151,6 @@ Do not use DNS for hostname lookup during installation .TP \fB\-\-no\-dns\-sshfp\fR Do not automatically create DNS SSHFP records. -.TP -\fB\-\-no\-serial\-autoincrement\fR -Do not enable SOA serial autoincrement feature. SOA serial will have to be updated automatically or other DNS features like zone transfer od DNSSEC will not function properly. This feature requires persistent search zone update mechanism. .SS "UNINSTALL OPTIONS" .TP diff --git a/ipaserver/install/bindinstance.py b/ipaserver/install/bindinstance.py index 2d4da087..6d5a1d44 100644 --- a/ipaserver/install/bindinstance.py +++ b/ipaserver/install/bindinstance.py @@ -463,7 +463,7 @@ class BindInstance(service.Service): def setup(self, fqdn, ip_address, realm_name, domain_name, forwarders, ntp, reverse_zone, named_user="named", zonemgr=None, - serial_autoincrement=True, ca_configured=None): + ca_configured=None): self.named_user = named_user self.fqdn = fqdn self.ip_address = ip_address @@ -474,7 +474,6 @@ class BindInstance(service.Service): self.suffix = ipautil.realm_to_suffix(self.realm) self.ntp = ntp self.reverse_zone = reverse_zone - self.serial_autoincrement = serial_autoincrement self.ca_configured = ca_configured if not zonemgr: @@ -590,9 +589,6 @@ class BindInstance(service.Service): else: ipa_ca = "" - def bool_to_yesno(var): - return "yes" if var else "no" - self.sub_dict = dict( FQDN=self.fqdn, IP=self.ip_address, @@ -605,7 +601,6 @@ class BindInstance(service.Service): OPTIONAL_NTP=optional_ntp, ZONEMGR=self.zonemgr, IPA_CA_RECORD=ipa_ca, - SERIAL_AUTOINCREMENT=bool_to_yesno(self.serial_autoincrement), ) def __setup_dns_container(self): |