summaryrefslogtreecommitdiffstats
path: root/install/tools/ipa-replica-install
diff options
context:
space:
mode:
authorMartin Kosek <mkosek@redhat.com>2011-10-06 11:20:53 +0200
committerMartin Kosek <mkosek@redhat.com>2011-10-06 11:28:25 +0200
commit526a39903a9d9136b7a2cef25cde59074ba1f39c (patch)
treec06cfbd834b3ead30ccf26e586c11a91b8a4c9dd /install/tools/ipa-replica-install
parent6dcd1d81216848b2428ea71237bcca826c40686d (diff)
downloadfreeipa-526a39903a9d9136b7a2cef25cde59074ba1f39c.tar.gz
freeipa-526a39903a9d9136b7a2cef25cde59074ba1f39c.tar.xz
freeipa-526a39903a9d9136b7a2cef25cde59074ba1f39c.zip
Improve ipa-replica-prepare DNS check
Currently, verify_fqdn() function raises RuntimeError for every problem with the hostname. This makes it difficult for tools like ipa-replica-prepare to behave differently for a subset of raised errors (for example to be able to create a DNS record for new replica when verify_fqdn() reports a lookup error). Implement own exceptions for verify_fqdn() that they can be safely used to distinguish the error type. https://fedorahosted.org/freeipa/ticket/1899
Diffstat (limited to 'install/tools/ipa-replica-install')
-rwxr-xr-xinstall/tools/ipa-replica-install4
1 files changed, 2 insertions, 2 deletions
diff --git a/install/tools/ipa-replica-install b/install/tools/ipa-replica-install
index c7ef54ce3..dbc736764 100755
--- a/install/tools/ipa-replica-install
+++ b/install/tools/ipa-replica-install
@@ -32,7 +32,7 @@ from ipaserver.install import bindinstance, httpinstance, ntpinstance, certs
from ipaserver.install.replication import replica_conn_check
from ipaserver.install.installutils import HostnameLocalhost, resolve_host
from ipaserver.install.installutils import ReplicaConfig, expand_replica_info, read_replica_info
-from ipaserver.install.installutils import get_host_name
+from ipaserver.install.installutils import get_host_name, BadHostError
from ipaserver.plugins.ldap2 import ldap2
from ipaserver.install import cainstance
from ipapython import version
@@ -331,7 +331,7 @@ def main():
config.dirman_password = dirman_password
try:
host = get_host_name(options.no_host_dns)
- except RuntimeError, e:
+ except BadHostError, e:
logging.error(str(e))
sys.exit(1)
if config.host_name != host: