diff options
author | Martin Kosek <mkosek@redhat.com> | 2011-12-07 13:58:35 +0100 |
---|---|---|
committer | Martin Kosek <mkosek@redhat.com> | 2011-12-07 14:45:09 +0100 |
commit | d040b93e386457266d5095bd696fa146962b640c (patch) | |
tree | f41607ad75e4c9925b2aeda4816fa2ab458eb815 | |
parent | 2203bb650ce4730832cb9799be0ea1b855c07170 (diff) | |
download | freeipa-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-x | install/tools/ipa-server-install | 25 |
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 |