URL: https://github.com/freeipa/freeipa/pull/3158 Author: rcritten Title: #3158: admintool: don't display log file on errors unless logging is setup Action: opened

PR body: """ The admintool will display the message when something goes wrong:

See %s for more information" % self.log_file_name

This is handy except when finally logging setup is not done yet so the log file doesn't actually get written to.

This can happen if validation catches and raises an exception.

The workaround is to save off and remove the log_file_name value before calling validation, then restore it if successful. This will suppress the above error message.

Fixes: https://pagure.io/freeipa/issue/7952

Signed-off-by: Rob Crittenden rcritten@redhat.com """

To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/3158/head:pr3158 git checkout pr3158