diff options
| author | François Cami <fcami@redhat.com> | 2019-07-29 11:22:09 +0200 |
|---|---|---|
| committer | Tibor Dudlák <tdudlak@redhat.com> | 2019-07-30 12:01:27 +0200 |
| commit | 402246a72990f77591f5efaec7ff0bc0cf82f416 (patch) | |
| tree | d055baf4bdd7f1e497ff5b33d828d1c2447a03ad /ipapython | |
| parent | 11e40336c5f0c5b086bbb7a98fb5cd22c80b4df3 (diff) | |
| download | freeipa-402246a72990f77591f5efaec7ff0bc0cf82f416.tar.gz freeipa-402246a72990f77591f5efaec7ff0bc0cf82f416.tar.xz freeipa-402246a72990f77591f5efaec7ff0bc0cf82f416.zip | |
ipapython/admintool.py: use SERVER_NOT_CONFIGURED
Commit 9182917280a5c2590fa677729db54b38a9ac4d1f introduced
SUCCESS, SERVER_INSTALL_ERROR and SERVER_NOT_CONFIGURED to
deal with cases when server is not configured.
Actually use SERVER_NOT_CONFIGURED in log_failure instead of 2.
Related-to: https://pagure.io/freeipa/issue/6843
Signed-off-by: François Cami <fcami@redhat.com>
Reviewed-By: Alexander Bokovoy <abokovoy@redhat.com>
Reviewed-By: Rob Crittenden <rcritten@redhat.com>
Diffstat (limited to 'ipapython')
| -rw-r--r-- | ipapython/admintool.py | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/ipapython/admintool.py b/ipapython/admintool.py index 5aa8cf1e9..ededb28bf 100644 --- a/ipapython/admintool.py +++ b/ipapython/admintool.py @@ -317,9 +317,7 @@ class AdminTool: # ipa-server-install. return message = "The %s command failed." % self.command_name - if self.log_file_initialized and return_value != 2: - # magic value because this is common between server and client - # but imports are not straigthforward + if self.log_file_initialized and return_value != SERVER_NOT_CONFIGURED: message += " See %s for more information" % self.log_file_name logger.error('%s', message) |