summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMartin Kosek <mkosek@redhat.com>2011-12-07 13:58:35 +0100
committerMartin Kosek <mkosek@redhat.com>2011-12-07 14:45:09 +0100
commitd040b93e386457266d5095bd696fa146962b640c (patch)
treef41607ad75e4c9925b2aeda4816fa2ab458eb815
parent2203bb650ce4730832cb9799be0ea1b855c07170 (diff)
downloadfreeipa-d040b93e386457266d5095bd696fa146962b640c.tar.gz
freeipa-d040b93e386457266d5095bd696fa146962b640c.tar.xz
freeipa-d040b93e386457266d5095bd696fa146962b640c.zip
Ask for user confirmation in ipa-server-install
Summarize entered IPA server configuration so that user can examine it and stop the installation process in case of error. Before this patch, user had to know which question is the last one and check the values he entered in various places during the interactive wizard. https://fedorahosted.org/freeipa/ticket/1083
-rwxr-xr-xinstall/tools/ipa-server-install25
1 files changed, 19 insertions, 6 deletions
diff --git a/install/tools/ipa-server-install b/install/tools/ipa-server-install
index 57deec64f..7a2e2aa1d 100755
--- a/install/tools/ipa-server-install
+++ b/install/tools/ipa-server-install
@@ -792,12 +792,6 @@ def main():
if options.reverse_zone and not bindinstance.verify_reverse_zone(options.reverse_zone, ip):
sys.exit(1)
- print "The IPA Master Server will be configured with"
- print "Hostname: " + host_name
- print "IP address: " + ip_address
- print "Domain name: " + domain_name
- print ""
-
if not options.realm_name:
realm_name = read_realm_name(domain_name, options.unattended)
root_logger.debug("read realm_name: %s\n" % realm_name)
@@ -848,6 +842,25 @@ def main():
dns_forwarders = ()
root_logger.debug("will use dns_forwarders: %s\n" % str(dns_forwarders))
+ print
+ print "The IPA Master Server will be configured with:"
+ print "Hostname: %s" % host_name
+ print "IP address: %s" % ip_address
+ print "Domain name: %s" % domain_name
+ print "Realm name: %s" % realm_name
+ print
+
+ if options.setup_dns:
+ print "BIND DNS server will be configured to serve IPA domain with:"
+ print "Forwarders: %s" % ("No forwarders" if not dns_forwarders \
+ else ", ".join([str(ip) for ip in dns_forwarders]))
+ print "Reverse zone: %s" % ("No reverse zone" if options.no_reverse \
+ else reverse_zone)
+ print
+
+ if not options.unattended and not user_input("Continue to configure the system with these values?", False):
+ sys.exit("Installation aborted")
+
# Installation has started. No IPA sysrestore items are restored in case of
# failure to enable root cause investigation
installation_cleanup = False