summaryrefslogtreecommitdiffstats
path: root/install
diff options
context:
space:
mode:
authorMartin Kosek <mkosek@redhat.com>2012-12-04 16:51:17 +0100
committerMartin Kosek <mkosek@redhat.com>2012-12-06 08:00:27 +0100
commit152585e73141ae5485e677f36f7f47551b438bbb (patch)
tree7da53433b83a0bc56166b96800f4eb9ec877e935 /install
parent62e7053a12fdfc38ac7ed212529c716428c7d92b (diff)
downloadfreeipa.git-152585e73141ae5485e677f36f7f47551b438bbb.tar.gz
freeipa.git-152585e73141ae5485e677f36f7f47551b438bbb.tar.xz
freeipa.git-152585e73141ae5485e677f36f7f47551b438bbb.zip
Improve ipa-replica-prepare error message
When DNS zone/record manipulation commands fails for example due to a ValidationError, ipa-replica-prepapre reports a whole traceback which is difficult to read. Make sure our error error is more readable. https://fedorahosted.org/freeipa/ticket/3283
Diffstat (limited to 'install')
-rwxr-xr-xinstall/tools/ipa-replica-prepare21
1 files changed, 17 insertions, 4 deletions
diff --git a/install/tools/ipa-replica-prepare b/install/tools/ipa-replica-prepare
index f9d0eb64..274e8456 100755
--- a/install/tools/ipa-replica-prepare
+++ b/install/tools/ipa-replica-prepare
@@ -467,13 +467,26 @@ def main():
if reverse_zone is None and not options.no_reverse:
reverse_zone = bindinstance.get_reverse_zone_default(ip)
- add_zone(domain)
- add_fwd_rr(domain, name, ip_address)
+ try:
+ add_zone(domain)
+ except errors.PublicError, e:
+ sys.exit("Could not create forward DNS zone for the replica: %s" % e)
+
+ try:
+ add_fwd_rr(domain, name, ip_address)
+ except errors.PublicError, e:
+ sys.exit("Could not add forward DNS record for the replica: %s" % e)
if reverse_zone is not None:
print "Using reverse zone %s" % reverse_zone
- add_zone(reverse_zone)
- add_ptr_rr(reverse_zone, ip_address, replica_fqdn)
+ try:
+ add_zone(reverse_zone)
+ except errors.PublicError, e:
+ sys.exit("Could not create reverse DNS zone for the replica: %s" % e)
+ try:
+ add_ptr_rr(reverse_zone, ip_address, replica_fqdn)
+ except errors.PublicError, e:
+ sys.exit("Could not add reverse DNS record for the replica: %s" % e)
try:
if not os.geteuid()==0: