diff options
author | Martin Kosek <mkosek@redhat.com> | 2011-10-12 10:55:08 +0200 |
---|---|---|
committer | Rob Crittenden <rcritten@redhat.com> | 2011-10-12 00:50:22 -0400 |
commit | 17f247d6c2aef177c40a690f886b0773a88a6dfa (patch) | |
tree | 8862c27a8c011d074309a5213991a77286d85e01 /ipa-client/ipa-install/ipa-client-install | |
parent | 2aa63fe4a98f6bae755b4ede607adc2068103c42 (diff) | |
download | freeipa-17f247d6c2aef177c40a690f886b0773a88a6dfa.tar.gz freeipa-17f247d6c2aef177c40a690f886b0773a88a6dfa.tar.xz freeipa-17f247d6c2aef177c40a690f886b0773a88a6dfa.zip |
ipa-client-install hangs if the discovered server is unresponsive
Add a timeout to the wget call to cover a case when autodiscovered
server does not response to our attempt to download ca.crt. Let
user specify a different IPA server in that case.
https://fedorahosted.org/freeipa/ticket/1960
Diffstat (limited to 'ipa-client/ipa-install/ipa-client-install')
-rwxr-xr-x | ipa-client/ipa-install/ipa-client-install | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/ipa-client/ipa-install/ipa-client-install b/ipa-client/ipa-install/ipa-client-install index e40072fff..c97aee24b 100755 --- a/ipa-client/ipa-install/ipa-client-install +++ b/ipa-client/ipa-install/ipa-client-install @@ -797,7 +797,8 @@ def install(options, env, fstore, statestore): if ret == ipadiscovery.NOT_FQDN: print >>sys.stderr, "%s is not a fully-qualified hostname" % hostname return CLIENT_INSTALL_ERROR - if ret == ipadiscovery.NO_LDAP_SERVER or not ds.getDomainName(): + if ret in (ipadiscovery.NO_LDAP_SERVER, ipadiscovery.NOT_IPA_SERVER) \ + or not ds.getDomainName(): logging.debug("Domain not found") if options.domain: cli_domain = options.domain @@ -815,7 +816,8 @@ def install(options, env, fstore, statestore): cli_domain = ds.getDomainName() logging.debug("will use domain: %s\n", cli_domain) - if ret == ipadiscovery.NO_LDAP_SERVER or not ds.getServerName(): + if ret in (ipadiscovery.NO_LDAP_SERVER, ipadiscovery.NOT_IPA_SERVER) \ + or not ds.getServerName(): logging.debug("IPA Server not found") if options.server: cli_server = options.server |