diff options
author | Ana Krivokapic <akrivoka@redhat.com> | 2013-04-22 17:09:47 +0200 |
---|---|---|
committer | Martin Kosek <mkosek@redhat.com> | 2014-02-18 16:37:07 +0100 |
commit | f67268db6855738350481491119b9be29ba1f22d (patch) | |
tree | edec218330798b48ab4b1f1259368d7ce14d6ad2 | |
parent | 7e9838042d5833cb2d3d7bec3a86857d5ffc526d (diff) | |
download | freeipa.git-f67268db6855738350481491119b9be29ba1f22d.tar.gz freeipa.git-f67268db6855738350481491119b9be29ba1f22d.tar.xz freeipa.git-f67268db6855738350481491119b9be29ba1f22d.zip |
Improve error message on failed Kerberos authentication
When ipa client installation fails due to failed Kerberos authentication,
make sure that the message about the failed authentication is displayed last.
This makes it clear to the user that this was the reason for failed
installation.
https://fedorahosted.org/freeipa/ticket/3573
Reviewed-By: Petr Viktorin <pviktori@redhat.com>
-rwxr-xr-x | ipa-client/ipa-install/ipa-client-install | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ipa-client/ipa-install/ipa-client-install b/ipa-client/ipa-install/ipa-client-install index 0d6037bd..7cc0c339 100755 --- a/ipa-client/ipa-install/ipa-client-install +++ b/ipa-client/ipa-install/ipa-client-install @@ -2292,9 +2292,9 @@ def install(options, env, fstore, statestore): stdin=stdin, env=env) if returncode != 0: + print_port_conf_info() root_logger.error("Kerberos authentication failed") root_logger.info("%s", stdout) - print_port_conf_info() return CLIENT_INSTALL_ERROR elif options.keytab: join_args.append("-f") @@ -2306,10 +2306,10 @@ def install(options, env, fstore, statestore): raiseonerr=False) if returncode != 0: + print_port_conf_info() root_logger.error("Kerberos authentication failed " "using keytab: %s", options.keytab) root_logger.info("%s", stdout) - print_port_conf_info() return CLIENT_INSTALL_ERROR else: root_logger.error("Keytab file could not be found: %s" |